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.

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

fix(react/v8): Support lazy-loaded routes and components #15281

Open
wants to merge 1 commit into
base: v8
Choose a base branch
from
Open

fix(react/v8): Support lazy-loaded routes and components #15281

wants to merge 1 commit into from

Conversation

onurtemizkan
Copy link
Collaborator

@onurtemizkan onurtemizkan commented Feb 3, 2025 •
edited
Loading

Backports #15039 to v8 branch

Potentially fixes as it also fixes cross-usage of createBrowserRouter and useRoutes: #15279

@onurtemizkan
fix(react): Support lazy-loaded routes and components. (#15039)
60ac908 
Fixes: #15027

This PR adds support for lazily loaded components and routes inside
`Suspend` on react-router pageloads / navigations.
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 3, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 23.3 KB - -
@sentry/browser - with treeshaking flags 23.17 KB - -
@sentry/browser (incl. Tracing) 35.9 KB - -
@sentry/browser (incl. Tracing, Replay) 73.27 KB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 66.71 KB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 77.57 KB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 89.5 KB - -
@sentry/browser (incl. Feedback) 39.51 KB - -
@sentry/browser (incl. sendFeedback) 27.91 KB - -
@sentry/browser (incl. FeedbackAsync) 32.71 KB - -
@sentry/react 25.98 KB - -
@sentry/react (incl. Tracing) 38.71 KB +0.02% +5 B 🔺
@sentry/vue 27.58 KB - -
@sentry/vue (incl. Tracing) 37.75 KB - -
@sentry/svelte 23.46 KB - -
CDN Bundle 24.49 KB - -
CDN Bundle (incl. Tracing) 37.6 KB - -
CDN Bundle (incl. Tracing, Replay) 72.9 KB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 78.23 KB - -
CDN Bundle - uncompressed 71.92 KB - -
CDN Bundle (incl. Tracing) - uncompressed 111.52 KB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 225.78 KB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 238.88 KB - -
@sentry/nextjs (client) 38.96 KB - -
@sentry/sveltekit (client) 36.4 KB - -
@sentry/node 162.85 KB - -
@sentry/node - without tracing 99.14 KB - -
@sentry/aws-serverless 131.23 KB - -

View base workflow run

@onurtemizkan onurtemizkan mentioned this pull request Feb 3, 2025
Open
3 tasks
@onurtemizkan onurtemizkan changed the title fix(react): Support lazy-loaded routes and components. (#15039) fix(react/v8): Support lazy-loaded routes and components. (#15039) Feb 3, 2025
@mydea mydea changed the title fix(react/v8): Support lazy-loaded routes and components. (#15039) fix(react/v8): Support lazy-loaded routes and components Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@s1gr1d s1gr1d s1gr1d 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.
2 participants
@onurtemizkan @s1gr1d