self-hosted icon indicating copy to clipboard operation
self-hosted copied to clipboard

Support new Beta User Feedback feature in self-hosted

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

New User Feedback workflow and JS widget to capture feedback outside of runtime errors (event_id link).

This will allow us have the feature on the Sandbox. And single tenant.

Feature flags:

"organizations:user-feedback-ui",
"organizations:user-feedback-ingest",
"organizations:issue-platform",
"organizations:feedback-ingest",
"organizations:feedback-post-process-group",
"organizations:feedback-visible",

Documentation

Product: https://docs.sentry.io/product/user-feedback/ SDK: https://docs.sentry.io/platforms/javascript/user-feedback/

bruno-garcia avatar Dec 12 '23 17:12 bruno-garcia

Mandatory question: Do we need additional Sentry worker or Snuba consumer containers?

aldy505 avatar Dec 15 '23 14:12 aldy505

@bruno-garcia do you know the answer to this? I peeked into sentry and it doesn't look like any additional infra should be required but not entirely sure

hubertdeng123 avatar Dec 19 '23 00:12 hubertdeng123

I found nothing more than a feature flag that should be enabled. Weird thing is that it did not ingest any feedbacks.

SebastiaanKloos avatar Dec 19 '23 11:12 SebastiaanKloos

I can confirm, that after adding the feature flags, I can see the UI but nothing is ingested.

holtgrewe avatar Dec 23 '23 16:12 holtgrewe

Hello, please also add these features:

"organizations:issue-platform",
"organizations:feedback-ingest",
"organizations:feedback-post-process-group",
"organizations:feedback-visible",

let me know if this does not work!

JoshFerge avatar Jan 02 '24 21:01 JoshFerge

Hello, please also add these features:

"organizations:issue-platform",
"organizations:feedback-ingest",
"organizations:feedback-post-process-group",
"organizations:feedback-visible",

let me know if this does not work!

Thank you so much. This fixed it. Woohoo!

CleanShot 2024-01-02 at 23 14 38

SebastiaanKloos avatar Jan 02 '24 22:01 SebastiaanKloos

This is very useful. Modify the Sentry configuration file sentry.conf.py and add the corresponding configuration. finally restart sentry service. is worked

New User Feedback workflow and JS widget to capture feedback outside of runtime errors (event_id link).

This will allow us have the feature on the Sandbox. And single tenant.

Feature flags:

"organizations:user-feedback-ui",
"organizations:user-feedback-ingest",
"organizations:issue-platform",
"organizations:feedback-ingest",
"organizations:feedback-post-process-group",
"organizations:feedback-visible",

Documentation

Product: https://docs.sentry.io/product/user-feedback/ SDK: https://docs.sentry.io/platforms/javascript/user-feedback/

zpfhyj avatar Jan 16 '24 08:01 zpfhyj

Anyone getting this after adding the features flags?

image

schettn avatar Mar 21 '24 07:03 schettn

With the addition of these 6 flags, I can now see the user feedback tab. But it can't receive any data. The user feedback tab on the details page of a single issue also has no data.

I'm using sentry 24.1.0 right now. And sending feedback using ios sdk like this.

public static func uploadUserFeedback(text: String?, eventID: SentryId?) {
	let eventID = eventID ?? SentrySDK.capture(message: "User Feedback")
	let userFeedback = Sentry.UserFeedback(eventId: eventID)
	userFeedback.comments = text ?? "no comment"
	userFeedback.email = SystemInformation.getUniqueDeviceIdentifierAsString()
	if ModuleCoordinator.userAccountModule().hasLogin() {
		userFeedback.name = ModuleCoordinator.userAccountModule().getHeyboxId() ?? ""
	}
	SentrySDK.capture(userFeedback: userFeedback)
}

Here is the full feature list.

"organizations:discover",
"organizations:events",
"organizations:global-views",
"organizations:incidents",
"organizations:integrations-issue-basic",
"organizations:integrations-issue-sync",
"organizations:invite-members",
"organizations:metric-alert-builder-aggregate",
"organizations:sso-basic",
"organizations:sso-rippling",
"organizations:sso-saml2",
"organizations:performance-view",
"organizations:user-feedback-ui",
"organizations:user-feedback-ingest",
"organizations:advanced-search",
"organizations:issue-platform",
"organizations:feedback-ingest",
"organizations:feedback-post-process-group",
"organizations:feedback-visible",
"projects:custom-inbound-filters",
"projects:data-forwarding",
"projects:discard-groups",
"projects:plugins",
"projects:rate-limits",
"projects:servicehooks",

mlch911 avatar Apr 12 '24 11:04 mlch911

@mlch911 Hi, have you re-run ./install.sh script yet?

Also please make sure that every of the feature flag defined here already exists on your config, you might do a diff check for this: https://github.com/getsentry/self-hosted/blob/5199bb07f927ca0a4f9330fedec857d1ac0fe811/sentry/sentry.conf.example.py#L265-L292

aldy505 avatar Apr 12 '24 13:04 aldy505

@mlch911 Hi, have you re-run ./install.sh script yet?

Also please make sure that every of the feature flag defined here already exists on your config, you might do a diff check for this:

https://github.com/getsentry/self-hosted/blob/5199bb07f927ca0a4f9330fedec857d1ac0fe811/sentry/sentry.conf.example.py#L265-L292

Are all these flags required? image

mlch911 avatar Apr 12 '24 14:04 mlch911

@mlch911 Hi, have you re-run ./install.sh script yet?

Also please make sure that every of the feature flag defined here already exists on your config, you might do a diff check for this:

https://github.com/getsentry/self-hosted/blob/5199bb07f927ca0a4f9330fedec857d1ac0fe811/sentry/sentry.conf.example.py#L265-L292

Do we need to re run ./install.sh after changing the config file?

mlch911 avatar Apr 12 '24 14:04 mlch911

@mlch911 Hi, have you re-run ./install.sh script yet?

Also please make sure that every of the feature flag defined here already exists on your config, you might do a diff check for this:

https://github.com/getsentry/self-hosted/blob/5199bb07f927ca0a4f9330fedec857d1ac0fe811/sentry/sentry.conf.example.py#L265-L292

Do we need to re run ./install.sh after changing the config file?

Sometimes, when new flags are added, there are database migrations (both Postgres and Clickhouse) that has to be run that depends on a certain feature flags.

Also it's recommended to have up-to-date feature flags as shown on the repository.

aldy505 avatar Apr 13 '24 07:04 aldy505

From @drguthals

User Feedback is officially here.

Get direct feedback from users when they encounter a bug on your site. We’ll attach their comment with all the debugging context you could dream of, like replay events, error details, URL tags, and more.

User Feedback makes debugging easier and faster, AND helps you identify trends across your entire user base so that you can prioritize the issues causing the most friction.

To get all the details on User Feedback, check out our latest blog post: https://bit.ly/sentry-user-feedback

https://discord.com/channels/621778831602221064/621786584110596119/1238164781311987763

So... it's GA already now?

aldy505 avatar May 10 '24 10:05 aldy505

It is GA, so it should be available on Self Hosted too

bruno-garcia avatar Jun 24 '24 19:06 bruno-garcia

@bruno-garcia What are the features that have to be enabled in order to make it work? I have this as baseline: https://github.com/getsentry/self-hosted/blob/24.6.0/sentry/sentry.conf.example.py

michaelmairegger avatar Jun 25 '24 06:06 michaelmairegger

It is GA, so it should be available on Self Hosted too

@bruno-garcia Do you want the team to create the PR for it?

aldy505 avatar Jun 25 '24 08:06 aldy505

Current known issue about user feedback feature (I just encountered this a few moments ago) https://github.com/getsentry/sentry/issues/67412

aldy505 avatar Jun 25 '24 08:06 aldy505