Skip to content

chore: update react and mui - #1502

Open
0xda157 wants to merge 5 commits into
devfrom
dionkud/react-19
Open

chore: update react and mui#1502
0xda157 wants to merge 5 commits into
devfrom
dionkud/react-19

Conversation

@0xda157

@0xda157 0xda157 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

part of SYNTH-145

Note

Not updating to mui v7 because it includes changes that break our UI styling that would require more work, I'll do this in a follow up PR.


Before merging, ensure the following criteria are met:

  • All acceptance criteria outlined in the ticket are met.
  • Necessary test cases have been added and updated.
  • A feature toggle or safe disable path has been added (if applicable).
  • User-facing polish:
    • Ask: "Is this ready-looking?"
  • Cross-linking between Jira and GitHub:
    • PR links to the relevant Jira issue.
    • Jira ticket has a comment referencing this PR.

@0xda157
0xda157 requested review from a team as code owners August 13, 2026 19:36
@0xda157
0xda157 marked this pull request as draft August 13, 2026 20:56
placeholder="..."
defaultValue={gameData}
onInput={(e: React.ChangeEvent<HTMLInputElement>) => setGameData(e.target.value)}
onChange={(e: React.ChangeEvent<HTMLInputElement>) => setGameData(e.target.value)}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

would be great if someone with more experience with the simulation UI could test this.

@0xda157
0xda157 marked this pull request as ready for review August 13, 2026 21:41
@0xda157
0xda157 requested a review from rutmanz August 13, 2026 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant