Implement useSentryUser parameter in SentryFeedbackWidget#3180
Merged
Conversation
inti7ary
requested review from
buenaflor,
denrase and
stefanosiano
as code owners
August 18, 2025 16:03
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3180 +/- ##
==========================================
+ Coverage 87.66% 89.54% +1.87%
==========================================
Files 291 96 -195
Lines 9896 3442 -6454
==========================================
- Hits 8675 3082 -5593
+ Misses 1221 360 -861 ☔ View full report in Codecov by Sentry. |
Collaborator
|
@inti7ary Thx for controibuting, looks like we forgot about this one. 🤦 Looks great so far. I need to merge some CI fixes, after that we can rebase and continue here. |
Collaborator
|
@inti7ary Would you be so kind and merge in the current main and add a changelog entry? |
Contributor
Author
|
I’ve merged main and added the changelog entry |
denrase
approved these changes
Aug 20, 2025
denrase
left a comment
denrase
left a comment
Collaborator
There was a problem hiding this comment.
Nice, thx for the contribution!
@buenaflor We just have the e2e test failing and some formatting which i will fix after the merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📜 Description
Implement the
useSentryUserparameter to prefill the name and email fields inSentryFeedbackWidget.💡 Motivation and Context
The
SentryFeedbackOptionsclass has auseSentryUseroption, but it doesn’t seem to be implemented yet. This feature allows automatically populating the feedback form with the current Sentry user’s data.💚 How did you test it?
Tested manually and updated the test suite to cover the
useSentryUserbehavior.📝 Checklist
sendDefaultPiiis enabled🔮 Next steps