Skip to content

fix(monitors): Update tour carousel copy to clarify issue creation#113169

Open
souredoutlook wants to merge 1 commit intomasterfrom
fix/monitors-tour-copy
Open

fix(monitors): Update tour carousel copy to clarify issue creation#113169
souredoutlook wants to merge 1 commit intomasterfrom
fix/monitors-tour-copy

Conversation

@souredoutlook
Copy link
Copy Markdown
Contributor

Rewrites steps 3 and 4 of the monitors onboarding carousel to more clearly explain that Monitors create Issues, and that Alerts notify teams about those Issues. Previous copy was vague about the role of issue creation in the monitor-to-alert flow.

  • Step 3 title: 'Alerts notify your team' → 'Alerts notify your team of issues'
  • Step 3 body: clarifies that issues created by Monitors trigger Alerts
  • Step 4 body: reframes connecting Alerts around issue notification routing

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

Rewrites steps 3 and 4 of the monitors onboarding carousel to more clearly
explain that Monitors create Issues, and that Alerts notify teams about those
Issues. Previous copy was vague about the role of issue creation in the
monitor-to-alert flow.

- Step 3 title: 'Alerts notify your team' → 'Alerts notify your team of issues'
- Step 3 body: clarifies that issues created by Monitors trigger Alerts
- Step 4 body: reframes connecting Alerts around issue notification routing

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@souredoutlook souredoutlook requested a review from a team as a code owner April 16, 2026 14:52
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 16, 2026
Copy link
Copy Markdown
Member

@malwilley malwilley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to merge if you like the wording as-is, just have some thoughts

<Text>
{t(
'Alerts notify your team. Define who gets paged, when, and how — via Slack, email, PagerDuty, and more.'
'Issues created by Monitors trigger Alerts. Alerts define who gets paged, when, and how — via Slack, email, PagerDuty, and more.'
Copy link
Copy Markdown
Member

@malwilley malwilley Apr 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence makes me wonder "are there issues which are not created by monitors?" 🤔

could also word this the first sentence this way: Alerts are triggered by issue state changes or Issue state changes will trigger Alerts

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Alerts are triggered when issues are created or updated."

Comment thread static/app/components/workflowEngine/ui/alertsMonitorsShowcase.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants