Skip to content
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

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

Open
3 of 6 tasks
Tracked by #1617
JackHaeg opened this issue Jun 18, 2024 · 2 comments · May be fixed by #1677
Open
3 of 6 tasks
Tracked by #1617

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

JackHaeg opened this issue Jun 18, 2024 · 2 comments · May be fixed by #1677
Assignees
Labels

Comments

@JackHaeg
Copy link
Member

JackHaeg commented Jun 18, 2024

Overview

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

Action Items

  • 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:
      • Replace these standard HTML components with applicable MUI components.
      • Make a PR for this file

Resources

@JackHaeg JackHaeg added this to the Y - Excellent Level milestone Jun 18, 2024
@JackHaeg JackHaeg changed the title ./src/components/admin/dashboard/index.js Update HTML components to MUI: ./src/components/admin/dashboard/index.js Jun 18, 2024
@chukalicious chukalicious self-assigned this Jun 19, 2024
@trillium
Copy link
Member

trillium commented Aug 8, 2024

Edit

Spoke too soon, this component needs changes

@trillium trillium closed this as completed Aug 8, 2024
@trillium trillium reopened this Aug 8, 2024
@JackHaeg
Copy link
Member Author

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment