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

Incorrect redirect when searching for 'Let Members Create and Edit Alerts' setting. #73692

Open
kerenkhatiwada opened this issue Jul 2, 2024 · 3 comments · May be fixed by #73697 or #73707
Open

Incorrect redirect when searching for 'Let Members Create and Edit Alerts' setting. #73692

kerenkhatiwada opened this issue Jul 2, 2024 · 3 comments · May be fixed by #73697 or #73707

Comments

@kerenkhatiwada
Copy link
Member

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

In the settings search, I get an option to go to a link to "Let Members Create and Edit Alerts" that has a description "Allow members to create, edit, and delete alert rules by granting them the "alerts:write" scope, but it links to https://<org_slug>.sentry.io/settings/#alertsMemberWrite which redirects to the settings page generically.
Screenshot 2024-07-02 at 2 28 56 PM

Expected Result

Redirected to https://<org_slug>.sentry.io/settings/organization/

Actual Result

Redirect to https:/<org_slug>.sentry.io/settings/#alertsMemberWrite

Product Area

Settings - General

Link

No response

DSN

No response

Version

No response

@getsantry
Copy link
Contributor

getsantry bot commented Jul 2, 2024

Auto-routing to @getsentry/product-owners-alerts for triage ⏲️

@getsantry
Copy link
Contributor

getsantry bot commented Jul 2, 2024

Routing to @getsentry/product-owners-settings-general for triage ⏲️

@roggenkemper
Copy link
Member

Thanks for bringing this up - seems to be a problem with more settings than just the one you pointed out. ill try to fix this

@roggenkemper roggenkemper linked a pull request Jul 2, 2024 that will close this issue
scttcper added a commit that referenced this issue Jul 2, 2024
routes for some settings are defined aas `/settings/:orgId/` and for some reason normalizeUrl is sending users to the root settings page. Adjust the route to send them to the right page.

fixes #73692
@scttcper scttcper linked a pull request Jul 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
3 participants