React-based web application that enables users to visualize both property graph and RDF data and explore connections between data without having to write graph queries.
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.
[x] 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 #347
Addresses #348
Completes #311
Check List
[x] I confirm that my contribution is made under the terms of the Apache 2.0 license.
[x] I have run pnpm checks to ensure code compiles and meets standards.
[x] I have run pnpm test to check if all tests are passing.
[ ] I've covered new added functionality with unit tests if necessary.
Description
key
prop being set by object spreadingreact-dnd
needed some slight adjustmentsuseEffect
that fires way too oftenuseEffect
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.
Focus On
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
Check List
pnpm checks
to ensure code compiles and meets standards.pnpm test
to check if all tests are passing.