Skip to content

Conversation

@lhvy
Copy link
Contributor

@lhvy lhvy commented Jul 15, 2025

Description 🧾

Still a lot of work to be done. Peer checking and testing required after each step below:

Testing

To be done incrementally by dev + other team members, plus one large FE test at the end.

@lhvy lhvy self-assigned this Jul 15, 2025
@lhvy lhvy added the 📦 dependencies 🚛 Pull requests that update a dependency file label Jul 15, 2025
@lhvy lhvy changed the base branch from dev to server-rewrite August 27, 2025 03:17
@lhvy lhvy force-pushed the mui branch 4 times, most recently from 4dc569f to b50bd56 Compare August 27, 2025 07:01
@lhvy lhvy changed the title WIP: Upgrade to MUI 7 Upgrade to MUI 7 Aug 27, 2025
@lhvy lhvy marked this pull request as ready for review August 27, 2025 07:01
@lhvy lhvy requested a review from a team as a code owner August 27, 2025 07:01
@lhvy lhvy requested a review from a team as a code owner August 31, 2025 23:01
@lhvy
Copy link
Contributor Author

lhvy commented Sep 1, 2025

Frontend looks the same to me, but a second pair of eyes testing and comparing locally would be awesome as well thanks.

Copy link
Contributor

@mark-trann mark-trann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good stuff lgtm. will test ui before approving

@lhvy lhvy merged commit fcd6109 into server-rewrite Sep 4, 2025
3 checks passed
@lhvy lhvy deleted the mui branch September 4, 2025 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📦 dependencies 🚛 Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants