VRMS icon indicating copy to clipboard operation
VRMS copied to clipboard

change some elements to mui -- more changes impending

Open evanyang1 opened this issue 1 year ago • 3 comments

Fixes #1684 (ongoing)

What changes did you make and why did you make them ?

Change to Material UI components

evanyang1 avatar Jun 29 '24 01:06 evanyang1

I don't know how to deal with forms.

evanyang1 avatar Jun 29 '24 01:06 evanyang1

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 hackforla-mui-1564 development
git pull https://github.com/hackforla/VRMS.git mui-1564

github-actions[bot] avatar Jun 29 '24 01:06 github-actions[bot]

Hey @evanyang1

Thanks for the work on this PR so far

Re changing forms

What have you tried so far?

Some possible next steps:

trillium avatar Aug 05 '24 22:08 trillium

@trillium Given this is a draft PR from Evan and he is no longer on the team, should we close this PR and place this issue back in the prioritized backlog for someone else to pick up? I can see that Evan made some more changes to the PR later on, so perhaps having someone review this could be good. Let me know what you think!

JackHaeg avatar Feb 04 '25 04:02 JackHaeg

Per @trillium - this PR is fairly straight forward, and can be reviewed. Keeping PR open for review.

For the reviewer(s) - please take a close look at this PR and ensure that you not only review changes, but also check for missing MUI changes to HTML components. If there are edits required to this PR, we will likely close this PR, unassign Evan from the issue (as he is currently an inactive team member), and place the issue back in the prioritized backlog.

JackHaeg avatar Feb 06 '25 19:02 JackHaeg

Per @jng34 there are some remaining HTML components that have not been changed to MUI. Closing this PR and re-assigning issue to be worked on by @jng34

JackHaeg avatar Mar 18 '25 03:03 JackHaeg