getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
37.08k stars 3.98k forks source link

[User Feedback]: Alert not triggered for "crash report" Feedback API using an SDK #70041

Closed bruno-garcia closed 6 days ago

bruno-garcia commented 2 weeks ago

I was validating this user report: https://github.com/getsentry/sentry/issues/5055#issuecomment-2082085673 by turning on alerting on this test org and firing a new issue and feedback

image

var id = SentrySdk.CaptureMessage("test 4");
SentrySdk.CaptureUserFeedback(id, "test 4","test 4");

I confirm new issue alert is working, and I received an email for that: image

I also enabled EA for this org, since UF is Beta I wasn't sure we gated any feature by EA-orgs only.

getsantry[bot] commented 2 weeks ago

Routing to @getsentry/product-owners-user-feedback for triage ⏲️

jas-kas commented 2 weeks ago

cc/ @JoshFerge @rodolfoBee

bruno-garcia commented 2 weeks ago

Oh, could it be the EA flag is cached on Relay and only updates after 1 hour? And I didn't wait long enough?

Just a thought here to help folks debugging ^

Relates to:

bruno-garcia commented 1 week ago

Fixes are live, I tested and validated it! 🚀

rodolfoBee commented 1 week ago

Is it already live? My organisation has EA enabled but I do not see the option: https://dev-curumas.sentry.io/settings/projects/notjava/user-feedback-processing/

jas-kas commented 1 week ago
Screenshot 2024-05-07 at 12 26 33 PM

@rodolfoBee It's under SDK Set Up not Processing 😅

https://dev-curumas.sentry.io/settings/projects/notjava/user-feedback/

rodolfoBee commented 1 week ago

Why there is two user feedback options in the menu?

jas-kas commented 1 week ago

@rodolfoBee Let me speak to the team -- we will try to consolidate. I think they're separate at the moment bc some represent Processing and the others are part of SDK Set Up :/ It makes for a worse UX though

https://sentry.slack.com/archives/C05J714QPRR/p1715181664051439

jas-kas commented 6 days ago

@rodolfoBee We're fixing it -- https://github.com/getsentry/sentry/issues/70663 tracking here