8000 fixing missingSuspenseWithCSRBailout warnings with nextjs 15 bump by Gregory-Pereira · Pull Request #356 · instructlab/ui · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fixing missingSuspenseWithCSRBailout warnings with nextjs 15 bump #356

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 21, 2024
Merged

fixing missingSuspenseWithCSRBailout warnings with nextjs 15 bump #356

merged 1 commit into from
Nov 21, 2024

Conversation

Gregory-Pereira
Copy link
Collaborator

This should fix build warnings (link), ex:

#20 15.50  ⚠ Invalid next.config.js options detected: 
#20 15.51  ⚠     Unrecognized key(s) in object: 'missingSuspenseWithCSRBailout' at "experimental"
#20 15.51  ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
#20 15.67 Attention: Next.js now collects completely anonymous telemetry regarding usage.
#20 15.67 This information is used to shape Next.js' roadmap and prioritize features.
#20 15.67 You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
#20 15.67 https://nextjs.org/telemetry
#20 15.67 
#20 16.77    ▲ Next.js 15.0.3

@Gregory-Pereira
fixing missingSuspenseWithCSRBailout warnings with nextjs 15 bump
8b67181 
Signed-off-by: greg pereira <grpereir@redhat.com>
@Gregory-Pereira
Copy link
Collaborator Author

See relevant link: https://nextjs.org/docs/messages/missing-suspense-with-csr-bailout#possible-ways-to-fix-it

@Gregory-Pereira
Copy link
Collaborator Author

Tested, works fine

brents-pet-robot
brents-pet-robot approved these changes Nov 21, 2024 •
edited
Loading
View reviewed changes
nerdalert
nerdalert approved these changes Nov 21, 2024
View reviewed changes
Copy link
Member
@nerdalert nerdalert left a comment

Choose a reason for hiding this comment

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

LGTM
< 8E01 form class="js-comment-update" data-type="json" data-turbo="false" action="/instructlab/ui/pull/356/reviews/2452712083/update" accept-charset="UTF-8" method="post">

@nerdalert nerdalert merged commit c91c538 into instructlab:main Nov 21, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nerdalert nerdalert nerdalert approved these changes

@brents-pet-robot brents-pet-robot brents-pet-robot approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Gregory-Pereira @nerdalert @brents-pet-robot
0