Issue 62 user management components admin dashboard #66
+2,044
−169
Conversation
This file contains 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
…eatures - feat(admin): implement `UserManagement` components for managing schools, users, and teams - feat(admin): add paginated data table (`data-grid`) and multiple-insert form (`data-table-form`) - feat(ui): add `SelectRole` and `SelectSchool` components for dropdown functionality - fix(sidebar): resolve issue with active tab not working correctly - refactor: change file naming in `/components/ui/` to follow PascalCase convention - refactor: move pagination props to a separate file for better organization
- feat(sidebar): add "Admin Portal" link to sidebar for navigation to Django Admin Portal - refactor(admin): update function exports to use named and default exports - docs(admin): add JSDoc comments to User Management module related files
loklokyx
requested review from
yunho7687,
Julie-Salazar,
AarifLamat,
Chloeiw and
leonleerl
|
hi, could you please attach some screenshots about your work and resolve the conflict? fantastic job btw! |
|
@yunho7687 Oops, I always forgot to add the screenshots for the components on the page, I have now added to the pull request. I've also resolved the merge conflicts. |
- feat(ui): create `WaitingLoader` component to show loading state during requests - feat(layout): get layout based on login status and role. Add breadcrumb navigation links - feat(sidebar): add logo to navigate landing page, add logout button - fix(sidebar): add missing `SheetTitle` and `SheetDescription` in Sidebar - feat(api): add `/users/me` mock data API - style(ui): add no-scrollbar CSS for better layout appearance
|
Hi @yunho7687, I’ve now combined the topbar with the sidebar as planned. The layout should be much more cohesive now. I’ve attached updated screenshots in the pull request for you to review. Also, I’ve added the |
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.
Change Summary
Added UserManagement components for the Admin Dashboard, including a paginated data table and multiple-insert form for schools, users, and teams. Improved the sidebar with a link to the Django Admin Portal and refactored some export functions. Added JSDoc to UserManagement module files.
Change Form
Other Information
Please review the
exportchanges in the UserManagement module and the addedAdmin Portallink in the sidebar.Screenshots
Related issue