-
Notifications
You must be signed in to change notification settings - Fork 367
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: fix modifyRoutes
in ssr
#4219
Conversation
🦋 Changeset detectedLatest commit: 4e4557f The changes in this PR will be included in the next version bump. This PR includes changesets to release 211 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4219 +/- ##
==========================================
- Coverage 57.59% 56.37% -1.23%
==========================================
Files 672 662 -10
Lines 17794 17988 +194
Branches 3876 3930 +54
==========================================
- Hits 10249 10141 -108
- Misses 6934 7187 +253
- Partials 611 660 +49 ☔ View full report in Codecov by Sentry. |
Summary
🤖 Generated by Copilot at db4d89f
This pull request improves the
modifyRoutes
hook functionality for the@modern-js/runtime
package. It simplifies the code inrouterPlugin
, fixes the hook execution order in SSR, and updates the changelog accordingly.Details
🤖 Generated by Copilot at db4d89f
@modern-js/runtime
package with SSR fixes (link)modifyRoutes
hook to reassignroutes
variable inplugin-runtime
package (link, link)modifyRoutes
hook inplugin-runtime
package (link)Related Issue
Checklist
pnpm run change
.