Skip to content
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

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

Closed
bruno-garcia opened this issue May 1, 2024 · 9 comments

Comments

@bruno-garcia
Copy link
Member

bruno-garcia commented May 1, 2024

I was validating this user report: #5055 (comment) 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
Copy link
Contributor

getsantry bot commented May 1, 2024

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

@jas-kas
Copy link
Member

jas-kas commented May 1, 2024

cc/ @JoshFerge @rodolfoBee

@bruno-garcia
Copy link
Member Author

bruno-garcia commented May 1, 2024

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
Copy link
Member Author

Fixes are live, I tested and validated it! 🚀

@rodolfoBee
Copy link
Member

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
Copy link
Member

jas-kas commented May 7, 2024

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
Copy link
Member

Why there is two user feedback options in the menu?

@jas-kas
Copy link
Member

jas-kas commented May 8, 2024

@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
Copy link
Member

jas-kas commented May 10, 2024

@rodolfoBee We're fixing it -- #70663 tracking here

@jas-kas jas-kas closed this as completed May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants