Skip to content

Add Sentry crash-reporting opt-out - #211

Merged
IanVS merged 3 commits into
mainfrom
sentry-opt-out
Dec 20, 2024
Merged

Add Sentry crash-reporting opt-out#211
IanVS merged 3 commits into
mainfrom
sentry-opt-out

Conversation

@IanVS

@IanVS IanVS commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

Closes #200

Before After

@IanVS
IanVS requested review from jasikpark, johnmaguire and nbrownus and removed request for jasikpark December 20, 2024 17:27
@nbrownus

nbrownus commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

Do we need to tear anything down when disabling?

nm I see it

@IanVS

IanVS commented Dec 20, 2024

Copy link
Copy Markdown
Contributor Author

It won't re-enable if it's toggled on, that will only take effect when the app restarts. But it disables immediately when toggled off.

@IanVS
IanVS merged commit 3d250f4 into main Dec 20, 2024
@IanVS
IanVS deleted the sentry-opt-out branch December 20, 2024 17:36
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.

Investigate Sentry crash reporting opt-out

3 participants