Skip to content

Conversation

@jasonyuezhang
Copy link
Owner

Without this django will redirect to the main page on sentry.io/monitors


Copied from getsentry#101901
Original PR: getsentry#101901

@jasonyuezhang jasonyuezhang force-pushed the copy/pr-101901-malwilley/fix/monitors-web-url branch from 94863c3 to 5b4b48a Compare October 22, 2025 00:35
@jasonyuezhang jasonyuezhang force-pushed the copy/pr-101901-malwilley/fix/monitors-web-url branch from 5b4b48a to 0040b92 Compare October 22, 2025 00:36
@propel-test-bot
Copy link

Add Top-Level monitors Route to Django Routing

This PR adds a new top-level route for monitors to the Django URL configuration in src/sentry/web/urls.py. This change ensures that requests to /monitors/ are correctly routed to the react_page_view instead of being redirected to the main page, aligning the web routing with expected behavior for Sentry's Monitors page.

Key Changes

• Added top-level re_path(r"^monitors/", react_page_view, name="monitors") entry to the URL patterns in src/sentry/web/urls.py

Affected Areas

src/sentry/web/urls.py

This summary was automatically generated by @propel-code-bot

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