Description

• Updates React to v18
  • Changed root render calls to the new style
  • Fixed stricter type requirements
  • Moved off deprecated types
  • Fixed key prop being set by object spreading
• Updates all React components to latest
  • Mantine jumped two versions and needed adjustments
  • The drag and drop lib react-dnd needed some slight adjustments
• Fixed existing issue exposed by the React v18 upgrade of a runaway useEffect that fires way too often
  • Moved the logic in to a component that is conditionally shown so that the useEffect is no longer necessary. The state that was updated is set when the component is shown, which is all that is needed.

Validation

Essentially, nothing should change from a user's perspective. This is purely a regression test.

• Test within Sagemaker Notebook before merging

Focus On

• Modals
• File inputs
• Export from table (nodes and edges)
• Drag and drop (search results to graph, connection file to connections)

Review Suggestions

Given the size of this change set, I would recommend reviewing the code changes commit by commit. It may also be useful to enable the "hide whitespace" in GitHub.

Related Issues

• Addresses Update React components to latest #347
• Addresses Update React to v18 #348
• Completes [Feature Request] Update React version #311

Check List

• I confirm that my contribution is made under the terms of the Apache 2.0 license.
• I have run pnpm checks to ensure code compiles and meets standards.
• I have run pnpm test to check if all tests are passing.
• I've covered new added functionality with unit tests if necessary.