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

docs(examples): fix next-translate prod hydration error in layout #52653

Merged
merged 4 commits into from
Aug 15, 2024

Conversation

aralroca
Copy link
Contributor

Related with aralroca/next-translate#1094

In development there aren't any error, but doing yarn build && yarn start was an hydration error. After check client and server HTML I realize that the server doesn't has the <html>, <head> tags, but the client HTML does. To fix it, I updated the layout, after this now is working fine.

It is likely that parallel to this error it would be good to notify during the development of this error if there is no <html> in the root layout. What do you think? I can look into making another PR to warn about this.

@aralroca aralroca requested review from a team as code owners July 13, 2023 15:46
@ijjk ijjk added the examples Issue was opened via the examples template. label Jul 13, 2023
@aralroca aralroca changed the title docs(examples): fix prod hydration error in layout docs(examples): fix next-translate prod hydration error in layout Jul 13, 2023
examples/with-next-translate/app/layout.js Outdated Show resolved Hide resolved
examples/with-next-translate/app/layout.js Outdated Show resolved Hide resolved
@leerob leerob merged commit 2865e18 into vercel:canary Aug 15, 2024
37 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue was opened via the examples template. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants