Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.8k
Feat: Improved error handling and response status availability#2303
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
vercelbot commentedNov 14, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
The latest updates on your projects. Learn more aboutVercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM.
Uh oh!
There was an error while loading.Please reload this page.
Koooooo-7 left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM, that really a customize handler hook enhancement.
Uh oh!
There was an error while loading.Please reload this page.
Summary
notFoundPageerror handlingSee#2294 for details.
Screenshots
Custom error handling via plugins (example)
Default error content updates (500 error)
Before: Displays inaccurate error information (shows 404 but should be 500)
After: Displays accurate error information
Initial render with content load error
Before: No sidebar, plugins halted
After: Sidebar rendered, plugins complete
Rendering empty files/routes
Before: Empty files incorrectly treated as 404 errors
After: Empty files rendered properly
Related issue, if any:
#2294
What kind of change does this PR introduce?
Feature
For any code change,
Does this PR introduce a breaking change?
No
Tested in the following browsers: