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

fix: ssr error handle #6278

Merged
merged 2 commits into from
May 29, 2023
Merged

fix: ssr error handle #6278

merged 2 commits into from
May 29, 2023

Conversation

chenjun1011
Copy link
Collaborator

reject error 后需要 return,避免进入后续逻辑

@chenjun1011 chenjun1011 added the need review Need Review label May 29, 2023
answershuto
answershuto previously approved these changes May 29, 2023
@@ -160,6 +160,7 @@ export async function renderToResponse(requestContext: ServerContext, renderOpti
onShellError: async (err) => {
if (renderOptions.disableFallback) {
reject(err);
return;
Copy link
Collaborator

Choose a reason for hiding this comment

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

changelog

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (release/next@94e7dff). Click here to learn what that means.
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@               Coverage Diff               @@
##             release/next    #6278   +/-   ##
===============================================
  Coverage                ?   80.52%           
===============================================
  Files                   ?      226           
  Lines                   ?    20597           
  Branches                ?     2611           
===============================================
  Hits                    ?    16586           
  Misses                  ?     3968           
  Partials                ?       43           
Impacted Files Coverage Δ
packages/runtime/src/runServerApp.tsx 78.18% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ClarkXia ClarkXia merged commit d1df9ff into release/next May 29, 2023
@ClarkXia ClarkXia deleted the fix-ssr-error-handle branch May 29, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need review Need Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants