feat(feedback): Return the eventId into the onSubmitSuccess callback

[ryan953]

Fixes #16836

[github-actions]

size-limit report 📦

Path	Size	% Change	Change
@sentry/browser	23.99 kB	-	-
@sentry/browser - with treeshaking flags	23.76 kB	-	-
@sentry/browser (incl. Tracing)	39.59 kB	-	-
@sentry/browser (incl. Tracing, Replay)	77.78 kB	-	-
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags	70.86 kB	-	-
@sentry/browser (incl. Tracing, Replay with Canvas)	82.5 kB	-	-
@sentry/browser (incl. Tracing, Replay, Feedback)	94.67 kB	+0.02%	+14 B 🔺
@sentry/browser (incl. Feedback)	40.76 kB	+0.03%	+10 B 🔺
@sentry/browser (incl. sendFeedback)	28.7 kB	-	-
@sentry/browser (incl. FeedbackAsync)	33.59 kB	+0.01%	+3 B 🔺
@sentry/react	25.76 kB	-	-
@sentry/react (incl. Tracing)	41.59 kB	-	-
@sentry/vue	28.37 kB	-	-
@sentry/vue (incl. Tracing)	41.37 kB	-	-
@sentry/svelte	24.01 kB	-	-
CDN Bundle	25.5 kB	-	-
CDN Bundle (incl. Tracing)	39.61 kB	-	-
CDN Bundle (incl. Tracing, Replay)	75.59 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback)	81.06 kB	+0.01%	+5 B 🔺
CDN Bundle - uncompressed	74.5 kB	-	-
CDN Bundle (incl. Tracing) - uncompressed	117.66 kB	-	-
CDN Bundle (incl. Tracing, Replay) - uncompressed	231.96 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed	244.79 kB	+0.01%	+8 B 🔺
@sentry/nextjs (client)	43.23 kB	-	-
@sentry/sveltekit (client)	40.05 kB	-	-
@sentry/node	162.02 kB	-	-
@sentry/node - without tracing	98.64 kB	+0.01%	+1 B 🔺
@sentry/aws-serverless	124.4 kB	+0.01%	+1 B 🔺

View base workflow run

[ryan953]

Ok, so it turns out that this is the eventId and things are working great.

But to make a url we instead support the issueID... why that's the case is a backend technical detail of how we built feedback. So before I can merge this I will need to support eventId in urls in addition to eventId.

Urls against sentry.io are something like this:

• https://<org-slug>.sentry.io/issues/feedback/?feedbackSlug=<project-slug>%3A<issue-id> (existing)
• https://<org-slug>.sentry.io/issues/feedback/?feedbackSlug=<project-slug>%3A<event-id> (todo)

we don't know the <org-slug> or <project-slug> so we can't make either url from within the SDK. But it should be trivial inside of onSubmitSuccess to apply the 2nd template.

[ryan953]

Aha! we already do support going from an eventId to a feedback in the url. The url template to use is this:
https://<org-slug>.sentry.io/issues/feedback/?projectSlug=<project-slug>&eventId=<event-id>

[ryan953]

To round it out we've got an update to the docs so others will be able to make use of the eventID.

That new param will be available in v10 of the javascript SDK, and all framework specific flavors too.