Skip to content

feat(feedback): Add Feedback SDK instructions to README #9351

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

Merged
merged 8 commits into from
Oct 26, 2023

Conversation

billyvg
Copy link
Member

@billyvg billyvg commented Oct 24, 2023

No description provided.

@billyvg billyvg marked this pull request as ready for review October 24, 2023 20:05
Comment on lines +18 to +19
During the alpha phase, the feedback integration will need to be imported from `@sentry-internal/feedback`. This will be
changed for the general release.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
During the alpha phase, the feedback integration will need to be imported from `@sentry-internal/feedback`. This will be
changed for the general release.
During the alpha phase, the feedback integration will need to be imported from `@sentry-internal/feedback`. This import will be
changed for the general release.

@@ -11,28 +11,33 @@ This SDK is **considered experimental and in an alpha state**. It may experience

## Pre-requisites

`@sentry/feedback` currently can only be used by browsers with [Shadow DOM](https://developer.mozilla.org/en-US/docs/Web/API/Web_components/Using_shadow_DOM) support.
`@sentry-internal/feedback` currently can only be used by browsers with [Shadow DOM](https://developer.mozilla.org/en-US/docs/Web/API/Web_components/Using_shadow_DOM) support.
Copy link
Member

Choose a reason for hiding this comment

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

I hate adding the ie11 note but people will def ask.

Suggested change
`@sentry-internal/feedback` currently can only be used by browsers with [Shadow DOM](https://developer.mozilla.org/en-US/docs/Web/API/Web_components/Using_shadow_DOM) support.
`@sentry-internal/feedback` currently can only be used by browsers with [Shadow DOM](https://developer.mozilla.org/en-US/docs/Web/API/Web_components/Using_shadow_DOM) support. This means it is not supported for IE11. Please see the [caniuse](https://caniuse.com/shadowdomv1) for the full browser compatability matrix.

Connect your own feedback UI to Sentry's You can use `feedback.flush()` to immediately send all currently captured feedback data.
When Feedback is currently in buffering mode, this will send up to the last 60 seconds of feedback data,
and also continue sending afterwards, similar to when an error happens & is recorded.
You can also bring your own widget and UI and simply pass a feedback object to the `sendFeedback()` function.
Copy link
Member

Choose a reason for hiding this comment

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

Might be nice for us to think about a way to make the widget work without js in the future via direct form submit, but def something for after GA (we can prob just open a feature request issue and see how many thumbs ups we get).

@billyvg billyvg merged commit 8b7b81e into develop Oct 26, 2023
@billyvg billyvg deleted the feat-replay-update-readme-for-alpha-instructions branch October 26, 2023 17:45
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.

3 participants