VRMS icon indicating copy to clipboard operation
VRMS copied to clipboard

Update HTML components to MUI: ./src/components/admin/dashboard/index.js

Open JackHaeg opened this issue 1 year ago • 2 comments

Overview

Review all components in./src/components/admin/dashboard/index.js and replace all standard HTML components with applicable MUI components.

Action Items

  • [x] Review components in:./src/components/admin/dashboard/index.js and search for any standard HTML components.
    • IF all components are already updated to relevant MUI
      • [ ] Comment on this issue that all components are already updated to MUI.
      • [ ] Close the issue & check the box associated with this issue on the MUI issue epic
      • [ ] Move on to the next MUI issue
    • IF standard HTML components are found in the file:
      • [x] Replace these standard HTML components with applicable MUI components.
      • [x] Make a PR for this file

Resources

  • https://mui.com/base-ui/all-components/
  • If you run into any questions, add a comment to the issue and tag "@trillium"

JackHaeg avatar Jun 18 '24 00:06 JackHaeg

Edit

Spoke too soon, this component needs changes

trillium avatar Aug 08 '24 16:08 trillium

@chukalicious Moved this issue back to "in progress" as this issue is still in progress, and has an active PR that is currently under review. Once the PR is merged, the issue will close automatically.

JackHaeg avatar Aug 13 '24 01:08 JackHaeg

Update:

  • PR closed: https://github.com/hackforla/VRMS/pull/1677, as the PR was out of date and included a file extension (.js instead of .jsx) that was incompatible with our current repo.
    • Issue has been added back to prioritized backlog to be picked up by a developer

JackHaeg avatar Jun 17 '25 03:06 JackHaeg