Skip to content

feat(crons): Allow monitors to be sorted by muted#64476

Merged
wedamija merged 1 commit into
masterfrom
danf/crons-monitors-sort-muted
Feb 5, 2024
Merged

feat(crons): Allow monitors to be sorted by muted#64476
wedamija merged 1 commit into
masterfrom
danf/crons-monitors-sort-muted

Conversation

@wedamija

@wedamija wedamija commented Feb 2, 2024

Copy link
Copy Markdown
Member

This allows monitors to be sorted by muted status. The logic for muted goes:

  • Muted on the monitor level comes first
  • Muted on any monitor environment comes second
  • If not muted, sort by name

@wedamija wedamija requested a review from a team as a code owner February 2, 2024 19:23
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 2, 2024
@wedamija wedamija enabled auto-merge (squash) February 2, 2024 19:27
@wedamija wedamija disabled auto-merge February 3, 2024 00:38
This allows monitors to be sorted by muted status. The logic for muted goes:
 - Muted on the monitor level comes first
 - Muted on any monitor environment comes second
 - If not muted, sort by name
@wedamija wedamija force-pushed the danf/crons-monitors-sort-muted branch from baa0fa6 to 227f310 Compare February 5, 2024 18:27
@wedamija wedamija merged commit 2a82cee into master Feb 5, 2024
@wedamija wedamija deleted the danf/crons-monitors-sort-muted branch February 5, 2024 18:57
cmanallen pushed a commit that referenced this pull request Feb 7, 2024
This allows monitors to be sorted by muted status. The logic for muted
goes:
 - Muted on the monitor level comes first
 - Muted on any monitor environment comes second
 - If not muted, sort by name
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants