Skip to content

Conversation

@jasonyuezhang
Copy link
Owner

was previously toggling the connect monitors drawer


Copied from getsentry#102115
Original PR: getsentry#102115

@propel-test-bot
Copy link

propel-test-bot bot commented Oct 26, 2025

Route Create New Monitor button to dedicated monitor creation page

Replaces the previous behavior where clicking Create New Monitor merely toggled the monitor-selection drawer. The button now uses LinkButton with a computed href from makeMonitorCreatePathname, sending the user directly to the monitor creation flow in a new tab.

Key Changes

• Added LinkButton import and replaced the former Button component for Create New Monitor
• Computed URL with makeMonitorCreatePathname(organization.slug, monitorsLinkPrefix) and marked it external
• Injected useMonitorViewContext and useOrganization to obtain monitorsLinkPrefix and organization.slug
• Removed obsolete on-click handler that opened the connect-monitors drawer

Affected Areas

static/app/views/automations/components/editConnectedMonitors.tsx (UI routing logic)

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

@jasonyuezhang jasonyuezhang force-pushed the mia/aci/fix-create-monitor branch from d8c34b3 to 273fc19 Compare October 27, 2025 08:44
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