Skip to content

Conversation

zoran995
Copy link
Collaborator

@zoran995 zoran995 commented Jun 28, 2025

Upgrade React to version 18. The build is failing since the terriajs release is not made yet

@zoran995 zoran995 requested a review from na9da June 30, 2025 15:36
Copy link
Contributor

@na9da na9da left a comment

Choose a reason for hiding this comment

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

Looks good @zoran995.
Just a few minor comments.

ReactDOM.render(
const container = document.getElementById("ui");
const root = createRoot(container);

Copy link
Contributor

Choose a reason for hiding this comment

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

Would be good to leave a comment here as to why we need flushSync

version "22.15.32"
resolved "https://registry.yarnpkg.com/@types/node/-/node-22.15.32.tgz#c301cc2275b535a5e54bb81d516b1d2e9afe06e5"
integrity sha512-3jigKqgSjsH6gYZv2nEsqdXfZqIFGAV36XYYjf9KGZ3PSG+IhLecqPnI310RvjutyMwifE2hhhNEklOUrvx/wA==
version "22.13.4"
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to downgrade the version, is that intentional?

@zoran995 zoran995 merged commit f7e7933 into main Jul 8, 2025
6 checks passed
@zoran995 zoran995 deleted the react-18-upgrade branch July 8, 2025 06:46
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.

3 participants