sentry-java icon indicating copy to clipboard operation
sentry-java copied to clipboard

Support for captureFeedback function

Open bruno-garcia opened this issue 1 year ago • 3 comments

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 (https://github.com/getsentry/sentry/issues/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 an event_id. The new function optionally takes a related event_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:

  • https://develop.sentry.dev/application/feedback-architecture/

This new function will allow outgoing feedbacks to be tied to Session Replay, which is currently Open Beta and we're working on GA:

  • https://github.com/getsentry/sentry/issues/74441

The develop docs have not been added yet, but the ticket with details about it is here:

  • https://github.com/getsentry/develop/issues/1206

Relates to:

  • https://github.com/getsentry/sentry-cocoa/issues/4187
  • https://github.com/getsentry/sentry-react-native/issues/3859
  • https://github.com/getsentry/sentry-go/issues/842
  • https://github.com/getsentry/sentry-dart/issues/1593
  • https://github.com/getsentry/sentry-javascript/issues/10953

bruno-garcia avatar Jul 23 '24 16:07 bruno-garcia

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/ Docs page with supported SDKs: https://docs.sentry.io/product/user-feedback/setup/#supported-sdks-for-user-feedback-widget Configuration page: https://docs.sentry.io/platforms/javascript/user-feedback/configuration/

markushi avatar Jul 31 '24 12:07 markushi

@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 avatar Oct 06 '24 05:10 bruno-garcia

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

markushi avatar Oct 08 '24 19:10 markushi

That's awesome! Now we gotta get @kahest 's 👀 Maybe something for Q1 2024 if it's that low effort?

bruno-garcia avatar Dec 28 '24 00:12 bruno-garcia

converted to Linear project

kahest avatar May 07 '25 17:05 kahest