-
-
Notifications
You must be signed in to change notification settings - Fork 46.9k
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: update workarounds for sub-components with Next.js App router #46711
docs: update workarounds for sub-components with Next.js App router #46711
Conversation
Signed-off-by: Navid <navidmafi2006@gmail.com>
Run & review this pull request in StackBlitz Codeflow. |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit dc6a10d:
|
…router Signed-off-by: Navid <navidmafi2006@gmail.com>
Added zh-CN translation using Machine translation (DeepL). Please let me know if any part is wrongly translated. |
Signed-off-by: Amumu <yoyo837@hotmail.com>
Signed-off-by: Amumu <yoyo837@hotmail.com>
Signed-off-by: Amumu <yoyo837@hotmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #46711 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 721 708 -13
Lines 12879 12122 -757
Branches 3427 3220 -207
==========================================
- Hits 12879 12122 -757 ☔ View full report in Codecov by Sentry. |
Signed-off-by: lijianan <574980606@qq.com>
Signed-off-by: lijianan <574980606@qq.com>
Signed-off-by: lijianan <574980606@qq.com>
Signed-off-by: lijianan <574980606@qq.com>
Signed-off-by: lijianan <574980606@qq.com>
Signed-off-by: lijianan <574980606@qq.com>
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
workaround for vercel/next.js#51593
💡 Background and solution
Using subcomponents of components (like
Typography
in Next.js app router causes an error that tells that client components are not dottable on the server.The current proposed fix is just to make the page fully client-rendered which works well as a workaround but is less than optimal since by making the component fully client-rendered developers can't use SSR patterns if they want to.
The new workaround that I'm proposing here is to extract the needed subcomponents into client components which allows for usage is SSR pages.
📝 Changelog
☑️ Self-Check before Merge