-
-
Notifications
You must be signed in to change notification settings - Fork 435
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
Support for captureFeedback function #3599
Comments
If we decide to build the widget using Jetpack Compose, we should move it into a separate Gradle module to avoid any dependency issues. Product docs: https://docs.sentry.io/product/user-feedback/ |
@markushi any idea of effort to build such widget? @armcknight is working on one for iOS and I wonder if we coudl announce them both together, if timelines aligned somehow. |
@bruno-garcia If we build the UI using Jetpack Compose, this should be doable within 2 days, as we can utilize the underlying theming feature and get that part for free. + 2 days for getting this released using craft 😅 |
Description
Description
This function is the base functionality needed to allow capturing user feedback while including tags,
replay_id
,trace_id
, etc.It's part of the 'new user feedback' product (getsentry/sentry#64671) that was shipped for Web. That included a built-in widget, that allows screenshot attachment, and under the hood relies on this new function that deprecates the old
captureUserFeedback
which required anevent_id
. The new function optionally takes a relatedevent_id
and will contain relevant context stand alone.This means a new envelope format for feedback was introduced.
The new Feedback product architecture has thoroughly documented by @aliu39 here:
This new function will allow outgoing feedbacks to be tied to Session Replay, which is currently Open Beta and we're working on GA:
The
develop
docs have not been added yet, but the ticket with details about it is here:Relates to:
captureFeedback
function sentry-cocoa#4187sendFeedback
tocaptureFeedback
& enhance it sentry-javascript#10953The text was updated successfully, but these errors were encountered: