-
Notifications
You must be signed in to change notification settings - Fork 359
feat(projects): migrate manage-projects module UI to Material UI (MUI) #1322
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
base: main
Are you sure you want to change the base?
Conversation
…age detection (final clean) chore: final eslint and format clean — ready for review
|
@Amitjoiya is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE. |
Avdhesh-Varshney
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uncomment manage-projects route from the App.tsx file and remove this route from the .vercelignore file.
When PR is ready to review, Attach the screenshots of these changes by accessing the respective route.
|
|
||
| axios | ||
| .post( | ||
| import.meta.env.VITE_SERVER_DOMAIN + '/api/project/delete', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All apis are already created in Frontend, checkout /infra/rest/apis folder
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plz take pull from the main branch as these changes are not acceptable.
|
@Amitjoiya Attach your respective issue with your PR |
|
@Amitjoiya Attach screenshots |







This PR migrates the manage-projects module UI from Tailwind to MUI as requested.
Included (Issue #1281 requirements)
✔️ Converted these files to MUI:
✔️ Uncommented manage-projects route in App.tsx and confirmed route is active
✔️ Removed manage-projects from .vercelignore
✔️ Attached screenshots of the route working
Additional necessary fixes (to make /dashboard/projects accessible):
These are NOT refactors — only minimal fixes required for the route to load:
sidebar/index.tsxto prevent premature redirect during auth initialization(This fixes the issue where
/dashboard/projectswas redirecting back to/.)Without these fixes, the manage-projects module could not be accessed even after UI migration.
No other files were changed.
Let me know if you want these routing fixes split into a separate PR.