Content-Length: 406850 | pFad | https://github.com/getsentry/sentry-docs/pull/14310

BD docs(feedback): Update docs for onSubmitSuccess to include eventID by ryan953 · Pull Request #14310 · getsentry/sentry-docs · GitHub
Skip to content

docs(feedback): Update docs for onSubmitSuccess to include eventID #14310

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

Merged
merged 3 commits into from
Jul 10, 2025

Conversation

ryan953
Copy link
Member

@ryan953 ryan953 commented Jul 9, 2025

Also added a note for how to use eventID to link back to sentry.io

SCR-20250709-ntqc

Related to getsentry/sentry-javascript#16835
Related to getsentry/sentry-javascript#16836

Copy link

vercel bot commented Jul 9, 2025

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

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 0:20am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
develop-docs ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2025 0:20am

@ryan953 ryan953 requested review from a team July 9, 2025 22:37
Copy link

codecov bot commented Jul 9, 2025

Bundle Report

Changes will increase total bundle size by 510 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-client-array-push 9.81MB -6 bytes (-0.0%) ⬇️
sentry-docs-server-cjs 11.9MB 516 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

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

Assets Changed:

Asset Name Size Change Total Size Change (%)
static/chunks/pages/_app-*.js -3 bytes 878.27kB -0.0%
static/chunks/1831-*.js -3 bytes 421.22kB -0.0%
static/wzAh4B2y3My0q23P8K5mK/_buildManifest.js (New) 684 bytes 684 bytes 100.0% 🚀
static/wzAh4B2y3My0q23P8K5mK/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/Q7KG9rHcCPVOc3XRzV4Lv/_buildManifest.js (Deleted) -684 bytes 0 bytes -100.0% 🗑️
static/Q7KG9rHcCPVOc3XRzV4Lv/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️
view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.67MB -0.0%
../instrumentation.js -3 bytes 997.28kB -0.0%
9523.js -3 bytes 973.22kB -0.0%
../app/[[...path]]/page.js.nft.json 175 bytes 702.42kB 0.02%
../app/platform-redirect/page.js.nft.json 175 bytes 702.33kB 0.02%
../app/sitemap.xml/route.js.nft.json 175 bytes 700.3kB 0.02%

ryan953 and others added 2 commits July 9, 2025 16:56
Co-authored-by: Michelle Zhang <56095982+michellewzhang@users.noreply.github.com>
Co-authored-by: Michelle Zhang <56095982+michellewzhang@users.noreply.github.com>
onSubmitError: (error: Error) => {},
}),
],
});
```

You can link directly to a saved feedback item by implementing the `onSubmitSuccess` callback. The URL you need to construct is `https://\$\{orgSlug}.sentry.io/issues/feedback/?projectSlug=\$\{projectSlug}&eventId=\$\{eventId}`.
Copy link
Member

Choose a reason for hiding this comment

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

do we need the escape chars?

Copy link
Member

Choose a reason for hiding this comment

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

idts

Copy link
Member Author

Choose a reason for hiding this comment

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

d'oh

Copy link
Member

Choose a reason for hiding this comment

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

🤖 autocomplete?

Copy link
Member Author

Choose a reason for hiding this comment

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

i was trying to make it a different template, which wasn't working. but i didn't clean up all this stuff.

Copy link
Member

@aliu39 aliu39 left a comment

Choose a reason for hiding this comment

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

Looks good, TIL we can use the eventId param in feedback

@ryan953 ryan953 merged commit e8b9f4f into master Jul 10, 2025
13 checks passed
@ryan953 ryan953 deleted the ryan953/feedback-onSuccess-eventId branch July 10, 2025 20:32
ryan953 added a commit that referenced this pull request Jul 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/getsentry/sentry-docs/pull/14310

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy