-
-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
change some elements to mui -- more changes impending #1689
base: development
Are you sure you want to change the base?
Conversation
I don't know how to deal with forms. |
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.
|
Hey @evanyang1 Thanks for the work on this PR so far Re changing formsWhat have you tried so far? Some possible next steps:
|
@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! |
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. |
Fixes #1684 (ongoing)
What changes did you make and why did you make them ?
Change to Material UI components