Skip to content

Conversation

@evanpurkhiser
Copy link
Member

@evanpurkhiser evanpurkhiser commented Nov 19, 2025

Part of NEW-564: There needs to be some way to mute the entire cron detector

Changes the monitor update flow to invalidate the query cache instead of manually updating cached data when muting/unmuting monitor environments. This ensures the UI always reflects the latest environment data from the server, including mute status and other environment properties.

Part of [NEW-564: There needs to be some way to mute the entire cron detector](https://linear.app/getsentry/issue/NEW-564/there-needs-to-be-some-way-to-mute-the-entire-cron-detector)

Changes the monitor update flow to invalidate the query cache instead of
manually updating cached data when muting/unmuting monitor environments.
This ensures the UI always reflects the latest environment data from the
server, including mute status and other environment properties.

- Simplified onUpdate to invalidate query instead of manually merging data
- Removed manual cache updates in DetailsTimeline
- Fixed mute action label to check actual environment mute status
- Cleaned up unused import
@evanpurkhiser evanpurkhiser requested review from a team as code owners November 19, 2025 22:15
@evanpurkhiser evanpurkhiser requested review from a team and removed request for a team November 19, 2025 22:15
@linear
Copy link

linear bot commented Nov 19, 2025

@evanpurkhiser evanpurkhiser requested a review from a team November 19, 2025 22:16
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 19, 2025
@evanpurkhiser evanpurkhiser enabled auto-merge (squash) November 19, 2025 22:19
? [
(env: string, isMuted: boolean) => ({
label:
isMuted && !monitor.isMuted
Copy link
Member Author

Choose a reason for hiding this comment

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

This is also needed to correctly set the title

@evanpurkhiser evanpurkhiser merged commit 4d473a1 into master Nov 19, 2025
51 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/ref-crons-invalidate-monitor-query-after-environment-muting branch November 19, 2025 22:58
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.

3 participants