Skip to content
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

SDK dev docs for User Feedback v2 #11635

Open
1 task
bruno-garcia opened this issue Oct 22, 2024 · 2 comments
Open
1 task

SDK dev docs for User Feedback v2 #11635

bruno-garcia opened this issue Oct 22, 2024 · 2 comments

Comments

@bruno-garcia
Copy link
Member

bruno-garcia commented Oct 22, 2024

Which section?

SDK -> Expected Feature -> User Feedback

Page URL

https://develop.sentry.dev/sdk/expected-features/#user-feedback

Description

Copying from the old repo, which got archived:

The current User Feedback SDK is documented here: https://develop.sentry.dev/sdk/features/#user-feedback

With the new feedback product allowing for capture of feedback outside errors. With the ability to include attachments (e.g: screenshots), tags, and other user context. The protocol has been expanded.

A new API is also being done on JS. Aligning on a new function captureFeedback. The previous captureUserFeedback can be deprecated in favor of the new.

The eventId that we took before is now optional. And the plan is to rename it to something else since the feedback itself will have an id, and eventId is ambiguous.

Document the new envelope format too. Mark the old one as deprecated.

This will unblock in particular the Mobile SDKs

Relates to:

Suggested Solution

Clarify the new API and guidelines

@aliu39
Copy link
Member

aliu39 commented Oct 22, 2024

The linked PR truncates messages to 4096 characters on Sentry backend, and doesn't error. It is not aware of bytes/encodings, since this is way after envelope processing. This is meant as a temporary measure before we add limits in the envelope schema, and dropped outcomes in Relay.

Until we have updates on the schema, I'd recommend documenting "4096 characters max per message".

@getsantry
Copy link
Contributor

getsantry bot commented Nov 7, 2024

Routing to @getsentry/product-owners-user-feedback for triage ⏲️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

2 participants