Skip to content

Conversation

tustanivsky
Copy link
Collaborator

@tustanivsky tustanivsky commented Aug 14, 2025

This PR reworks existing functionality that allows capturing user feedback.

Key changes:

  • USentryUserFeedback class was replaced with USentryFeedback to align with underlying SDKs naming
  • Feedback no longer needs to be associated with a specific event and the only required parameter is the user message
  • Updated related API in USentrySubsystem and USentryLibrary
  • On Windows and Linux, the string representation of an event ID no longer contains dashes unifying its format with other platforms
  • Added automation tests

Corresponding docs updates:

Related to:

Copy link
Contributor

github-actions bot commented Aug 14, 2025

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- New API for capturing user feedback ([#1051](https://github.com/getsentry/sentry-unreal/pull/1051))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against b66f96d

Copy link
Member

@bruno-garcia bruno-garcia left a comment

Choose a reason for hiding this comment

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

RSLGTM

@tustanivsky tustanivsky merged commit bb555d9 into main Aug 25, 2025
10 of 11 checks passed
@tustanivsky tustanivsky deleted the feat/new-feedback branch August 25, 2025 12:03
tustanivsky added a commit to getsentry/sentry-docs that referenced this pull request Aug 25, 2025
This PR updates code examples (both C++ and blueprint) showing how to
use the new user feedback API.

Related to:
- getsentry/sentry-unreal#1051
tustanivsky added a commit to getsentry/sentry-docs that referenced this pull request Aug 29, 2025
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.

4 participants