Skip to content

fix(docs): update sentry basics react tutorial - #12967

Closed
a-hariti wants to merge 4 commits into
masterfrom
fix-react-tutorial
Closed

fix(docs): update sentry basics react tutorial#12967
a-hariti wants to merge 4 commits into
masterfrom
fix-react-tutorial

Conversation

@a-hariti

@a-haritia-hariti commented Mar 10, 2025

Copy link
Copy Markdown
Collaborator

related #12918

todo:

  • update deps on the tutorial: PR
  • Edit these docs further to follow the instrument.js file convention from the React guide

@vercel

vercelBot commented Mar 10, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
sentry-docs✅ Ready (Inspect)Visit Preview💬 Add feedbackMar 18, 2025 4:05pm
2 Skipped Deployments
NameStatusPreviewCommentsUpdated (UTC)
changelog⬜️ Ignored (Inspect)Visit PreviewMar 18, 2025 4:05pm
develop-docs⬜️ Ignored (Inspect)Visit PreviewMar 18, 2025 4:05pm

@codecov

codecovBot commented Mar 10, 2025

Copy link
Copy Markdown

Bundle Report

Changes will decrease total bundle size by 15 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle nameSizeChange
sentry-docs-server-cjs10.53MB-9 bytes (-0.0%) ⬇️
sentry-docs-client-array-push9.39MB-6 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset NameSize ChangeTotal SizeChange (%)
static/chunks/pages/_app-*.js-3 bytes868.16kB-0.0%
static/chunks/3319-*.js-3 bytes393.41kB-0.0%
server/middleware-*.js-5.55kB1.0kB-84.74%
server/middleware-*.js5.55kB6.55kB555.3% ⚠️
static/pZ2VAlV-*.js(New)578 bytes578 bytes100.0% 🚀
static/pZ2VAlV-*.js(New)77 bytes77 bytes100.0% 🚀
static/B0l4g2zUTFzmbjyW4Lz7u/_buildManifest.js(Deleted)-578 bytes0 bytes-100.0% 🗑️
static/B0l4g2zUTFzmbjyW4Lz7u/_ssgManifest.js(Deleted)-77 bytes0 bytes-100.0% 🗑️
view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset NameSize ChangeTotal SizeChange (%)
1729.js-3 bytes1.57MB-0.0%
../instrumentation.js-3 bytes904.32kB-0.0%
9523.js-3 bytes880.37kB-0.0%

Comment threaddocs/product/sentry-basics/integrate-frontend/initialize-sentry-sdk.mdx Outdated
Comment threaddocs/product/sentry-basics/integrate-frontend/initialize-sentry-sdk.mdx Outdated

@coolguyzonecoolguyzone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for updating! 🤙🛳️

a-haritiand others added 2 commits March 17, 2025 16:22
…y-sdk.mdx
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…y-sdk.mdx
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
@a-hariti

Copy link
Copy Markdown
CollaboratorAuthor

@coolguyzone plz take a look again and please approve this PR

@coolguyzonecoolguyzone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wasn't able to successfully code along with the updates, it seems like the sample repo would need to be updated to support this, lmk what you think!

1. Install the Sentry React SDK using NPM.

Make sure you're in the `frontend-tutorial` project folder.
Open src/index.js where you will find the 'Sentry.init()' call after the last import statement. Paste the DSN key value you copied from the project created in the [previous section](https://github.com/product/sentry-basics/integrate-frontend/create-new-project/) into the call and save this file:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The file to open here should be src/instrument.js, correct?

@chargome

Copy link
Copy Markdown
Member

stale

@github-actionsgithub-actionsBot locked and limited conversation to collaborators Mar 8, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@a-hariti@chargome@coolguyzone