Skip to content

Conversation

inti7ary
Copy link
Contributor

@inti7ary inti7ary commented Aug 18, 2025

📜 Description

Implement the useSentryUser parameter to prefill the name and email fields in SentryFeedbackWidget.

💡 Motivation and Context

The SentryFeedbackOptions class has a useSentryUser option, 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 useSentryUser behavior.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
  • I updated the docs if needed
  • All tests passing
  • No breaking changes

🔮 Next steps

Copy link

codecov bot commented Aug 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.54%. Comparing base (05da225) to head (b861b1a).
⚠️ Report is 1 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

@denrase
Copy link
Collaborator

denrase commented Aug 19, 2025

@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.

@denrase
Copy link
Collaborator

denrase commented Aug 19, 2025

@inti7ary Would you be so kind and merge in the current main and add a changelog entry?

@inti7ary
Copy link
Contributor Author

I’ve merged main and added the changelog entry

Copy link
Collaborator

@denrase denrase left a comment

Choose a reason for hiding this comment

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

Nice, thx for the contribution!

@buenaflor We just have the e2e test failing and some formatting which i will fix after the merge.

@denrase denrase merged commit c1e775e into getsentry:main Aug 20, 2025
48 of 52 checks passed
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.

2 participants