VRMS
VRMS copied to clipboard
hackforla
Migrated edit users to Material UI
Fixes #1723
What changes did you make and why did you make them ?
- Migrated editUsers.js to Material UI
Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)
Visuals before changes are applied
Visuals after changes are applied
Want to review this pull request? Take a look at this documentation for a step by step guide!
From your project repository, check out a new branch and test the changes.
git checkout -b ntrehan-1723 development
git pull https://github.com/ntrehan/VRMS.git 1723
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
@trillium I see that you approved these changes on this PR. Can we go ahead and merge?
@trillium & @ntrehan:
- This PR was completed prior to the introduction of the VRMS Admin feature (that allows admins to promote/demote other admins).
- The VRMS Admin feature is currently implemented in Dev (see attached screenshot below)
Question:
- Will this PR result in the "VRMS Admin" component using MUI styling, or will it default to the previous styling?
- If it will result in the old styling or merging will be difficult with these new changes, should we either update this PR with this fix, or create a new issue to track this change entirely?
Screenshot of current Dev build with "VRMS Admin" feature
@trillium Please see my comment above and let me know your thoughts - https://github.com/hackforla/VRMS/pull/1727#issuecomment-2712448409