-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
docs(js): Review and update TanStack Start React quick start guide #14879
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
docs(js): Review and update TanStack Start React quick start guide #14879
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bundle ReportChanges will decrease total bundle size by 87.04kB (-0.37%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
|
| ``` | ||
|
|
||
| ### Capture Errors in your Error Boundaries and Components | ||
| ### Client-Side Error Boundaries (Optional) |
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.
I think we should not name this client side if we're talking about server side right below, wdyt? Maybe something more general
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.
Updated to "Capturing Errors in Error Boundaries and Components (Optional)" -- almost identical to the original title.
But please let me know if you have a more general idea :)
coolguyzone
left a comment
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.
Thanks, looks good! 😊
…ick-start/review-tanstack-start-react
DESCRIBE YOUR PR
Reviewed the existing TanStack Start React quick start guide and made some updates:
Closes: #14878
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
EXTRA RESOURCES
Note
Rewrites and restructures the TanStack Start React quick start with clearer client/server setup, SSR and server instrumentation, updated tracing example, and improved verification and links.
index.mdx):wrapCreateRootRouteWithSentryandwrapStreamHandlerWithSentry; grouped under performance option.sentryGlobalServerMiddlewareHandler.withErrorBoundaryand capturingerrorComponenterrors withcaptureException; note on automatic client capture.src/routes/sentry-example.ts(/sentry-example); updated fetch path and example code.config.yml): Updated description to a more comprehensive summary.Written by Cursor Bugbot for commit 2ba052c. This will update automatically on new commits. Configure here.