Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solved Hydration Issue #90

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RonakSurana-2001
Copy link

Solved the turbopack issue #10
Adding a video for verification purposes

Media1.mp4

Copy link
Member

@JeelRajodiya JeelRajodiya left a comment

Choose a reason for hiding this comment

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

Hello, While running the code I encountered these errors.

image
image

I also tried running the the same code in github codespaces (to ensure that these errors are not only specific to my machine)
While testing, I found that on the first run the site does not throw any error, but when I stop the server and restart it, these error appears. Please take a look

image
image

@RonakSurana-2001
Copy link
Author

RonakSurana-2001 commented Oct 5, 2024

Even I ran in github codespaces I not encountered any error. Even when I ran multiple times.
See the video.

Media1.2.mp4

@JeelRajodiya
Copy link
Member

JeelRajodiya commented Oct 5, 2024

I see, We are running different versions of next.js maybe because of that we have different outcomes.
Can you please upgrade the Next.js version on this PR?
Follow this guide: https://nextjs.org/docs/app/building-your-application/upgrading/version-14

@RonakSurana-2001
Copy link
Author

RonakSurana-2001 commented Oct 5, 2024

image

Its Next 14 already

@RonakSurana-2001
Copy link
Author

Maintainers please reply

@JeelRajodiya
Copy link
Member

@RonakSurana-2001
image

Please check your package.json file, the next.js version is specified there.

"next": "^14.2.5",

Copy link
Member

Choose a reason for hiding this comment

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

Are these changes relevant to the issue? if yes please leave an explanation about what you did.

Copy link
Member

Choose a reason for hiding this comment

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

Please leave an explanation about how this solves the issue? I suggest we avoid delayed rendering of the providers and their children, this can affect the server side build of the project and may cause performance issues.

@JeelRajodiya
Copy link
Member

JeelRajodiya commented Oct 7, 2024

Even I ran in github codespaces I not encountered any error. Even when I ran multiple times. See the video.

Media1.2.mp4

I see in your recording we have package-lock.json, which means you are installing packages using npm install. Instead of npm, please use pnpm install to install the packages, you can follow the guide from the README.MD

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.

2 participants