sentry icon indicating copy to clipboard operation
sentry copied to clipboard

ref(feedback): consolidate all settings into one heading under project

Open michellewzhang opened this issue 9 months ago • 3 comments

closes https://github.com/getsentry/sentry/issues/70663 as a fun bonus, this is also a deprecated async conversion so ref https://github.com/getsentry/frontend-tsc/issues/2

Before, we had two User Feedback headings in project settings, one under Processing (for spam detection) and one under SDK Setup (for crash report alerting options):

SCR-20240510-nqlf

This PR moves all 3 settings into one User Feedback header under the main Project heading:

SCR-20240510-nsig-2

michellewzhang avatar May 10 '24 19:05 michellewzhang

Slightly unrelated but I'm almost certain the "Show Sentry branding" is for the Crash-Report modal specifically. Lets also update the copy for that while we're at it?

jas-kas avatar May 10 '24 19:05 jas-kas

Otherwise, LGTM!

nyan

jas-kas avatar May 10 '24 19:05 jas-kas

Slightly unrelated but I'm almost certain the "Show Sentry branding" is for the Crash-Report modal specifically. Lets also update the copy for that while we're at it? SCR-20240510-nwff

updated!

michellewzhang avatar May 10 '24 19:05 michellewzhang

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: useNavigate() may be used only in the context of a <Router> component. useNavigate(useNavigate.tsx) View Issue

Did you find this useful? React with a 👍 or 👎

sentry-io[bot] avatar May 15 '24 20:05 sentry-io[bot]