Skip to content

feat(monitors): Add onboarding modal for the new alerts/monitors pages#112694

Merged
malwilley merged 5 commits intomasterfrom
malwilley/feature-tour-button
Apr 10, 2026
Merged

feat(monitors): Add onboarding modal for the new alerts/monitors pages#112694
malwilley merged 5 commits intomasterfrom
malwilley/feature-tour-button

Conversation

@malwilley
Copy link
Copy Markdown
Member

@malwilley malwilley commented Apr 10, 2026

Ref ISWF-2160

Uses the new FeatureShowcase component to add onboarding to the monitors and alerts index pages.

All svgs were run through svgo

CleanShot.2026-04-10.at.10.50.50.mp4

Replace the old FeatureTourModal-based tour with the new FeatureShowcase
compound component. The tour is now self-contained in the
WorkflowEngineFeatureTourButton with 4 steps covering Monitors, Alerts,
and how they connect together. Both the Alerts and Monitors list pages
share the same tour.

Also tweaks FeatureShowcase to use Stack instead of Container and adds
size="sm" to the close button for better alignment.
@malwilley malwilley requested review from a team as code owners April 10, 2026 17:53
@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 10, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 10, 2026
@malwilley malwilley removed request for a team April 10, 2026 17:54
import monitorsTourAlertsImage from 'sentry-images/spot/monitors-tour-alerts.svg';
import monitorsTourConnectingImage from 'sentry-images/spot/monitors-tour-connecting.svg';
import monitorsTourIntroImage from 'sentry-images/spot/monitors-tour-intro.svg';
import monitorsTourMonitorsImage from 'sentry-images/spot/monitors-tour-monitors.svg';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: should this graphic include mobile builds as well? (don't want to block the PR on this, just curious if we should try to get that in the graphic for launch)

Comment thread static/app/components/workflowEngine/workflowEngineFeatureTourButton.tsx Outdated
Comment thread static/app/components/workflowEngine/workflowEngineFeatureTourButton.tsx Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d024033. Configure here.

@malwilley malwilley merged commit a290948 into master Apr 10, 2026
70 of 72 checks passed
@malwilley malwilley deleted the malwilley/feature-tour-button branch April 10, 2026 19:47
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