feat/admin dashboard #20
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
Coderx85
commented
Jan 1, 2026
Coderx85
commented
- Restrucuture the components folder.
- Refactor imports and remove unused sidebar configuration files; add NotFound component and health check API route.
- Implement the members table.
- Refactor admin routes.
- Add organization settings page and permission checks for admin roles.
- Add admin dashboard components for meeting analytics and member management.
- Update loginAction to return organization existence status in response.
- Refactor admin dashboard routes and components; update Docker Hub token in CI workflow.
- feat: Implement custom participant view and layouts with role indicators and audio level visualization.
- feat: Implement workspace context sync, enhanced meeting participant actions, and audio level indicators.
- refactor: Reimplement meeting participant views using Stream SDK layouts with custom UI components and add a new database migration.
…otFound component and health check API route.
…en in CI workflow.
…ors and audio level visualization.
…actions, and audio level indicators.
…uts with custom UI components and add a new database migration.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR implements a comprehensive admin dashboard feature with significant restructuring of the component architecture and addition of meeting analytics and member management capabilities.
Key Changes:
- Added admin dashboard with meeting analytics (daily/weekly charts)
- Implemented meeting participants tracking with a new database table
- Refactored permission system to use organization-based permissions
- Added custom participant views with role indicators and audio visualization
- Restructured components folder with better organization
Reviewed changes
Copilot reviewed 90 out of 103 changed files in this pull request and generated 26 comments.
Show a summary per file
| File | Description |
|---|---|
src/types/database.ts |
Removed user role definitions, now imported from action types |
src/types/api.ts |
Added new API response types and workspace member types |
src/types/action.ts |
Expanded role types with admin and member specific types |
src/lib/permission.ts |
Refactored to use organization permissions instead of workspace |
src/hooks/useParticipantRole.ts |
New hook to fetch participant roles in meetings |
src/hooks/useGetCallsBySlug.ts |
New hook to fetch calls by workspace slug |
src/db/schema/schema.ts |
Added meeting participants table and meeting/participant status enums |
src/components/workspace/meeting/participant/* |
New custom participant view components with role badges |
src/components/workspace/admin/charts/* |
Admin dashboard analytics charts |
src/action/user.action.ts |
Updated login to return organization existence status |
src/action/participant.action.ts |
New action to get participant roles |
src/action/meeting.action.ts |
Added meeting access checks and participant management |
package.json |
Removed framer-motion, vitest dependencies; fixed "lin" typo to "lint" |
.github/workflows/publish.yaml |
Updated Docker Hub token secret name |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| }; | ||
|
|
||
| fetchRole(); | ||
| }, [participant.userId, participant.custom, participant.name]); |
There was a problem hiding this comment.
The dependency array is missing workspaceSlug which is used in the effect. This could cause stale data to be used if the workspace changes.
| meeting: ["create"], | ||
| // Members can create and view organizations but cannot delete or update | ||
| organization: ["create", "view"], | ||
| meeting: ["create", "update", "delete"], |
There was a problem hiding this comment.
Members are given "update" and "delete" permissions for meetings, but this seems overly permissive. Members should likely only be able to update/delete meetings they created, not all meetings. Consider restricting these permissions or implementing ownership checks.
…seeding, and enhance workspace store with slug.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.