Skip to content

Conversation

@nozomione
Copy link
Member

@nozomione nozomione commented Dec 17, 2025

Issue Number

N/A

Target branch: feature/dataset-integration

(This PR currently targets the dataset feature branch since the error is present. However, it does not affect QA testing and can be merged after the Dataset launch.)

Purpose/Implementation Notes

This PR addresses the following console error message generated on localhost for a better development experience:

Prop id did not match. Server: "8iqic292wa" Client: "gpkvfdwmha"
circle
defs
svg
Samples
...
See more info here: https://nextjs.org/docs/messages/react-hydration-error

Change includes:

  • Replaced the cleanupIDs plugin with run-time generated dynamical IDs (using useState and useEffect) to prevent the hydration error.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

Functional tests

N/A

Checklist

  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Screenshots

Screenshot 2025-12-17 at 2 14 02 PM

After the change:

e.g., For the "Guidelines" SVG, the id value is unique for each instance of the SVG:

Screenshot 2025-12-18 at 10 58 55 AM Screenshot 2025-12-18 at 10 58 35 AM

…cleanupIDs in babelrc to prevent hydration error
@nozomione nozomione self-assigned this Dec 17, 2025
@nozomione nozomione added the bug Something isn't working label Dec 17, 2025
@nozomione nozomione changed the base branch from feature/dataset-integration to dev December 24, 2025 21:56
@nozomione nozomione changed the base branch from dev to feature/dataset-integration December 24, 2025 21:58
@nozomione nozomione changed the base branch from feature/dataset-integration to dev December 25, 2025 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Low Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants