Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show 100 latest alerts on alert group page #1417

Merged
merged 4 commits into from
Feb 28, 2023
Merged

Conversation

vstpme
Copy link
Member

@vstpme vstpme commented Feb 27, 2023

What this PR does

Make internal API return 100 latest alerts for alert group.

Which issue(s) this PR fixes

#857

Checklist

  • Tests updated
  • CHANGELOG.md updated

@vstpme vstpme requested review from a team February 27, 2023 09:40
@vstpme vstpme added the pr:no public docs Added to a PR that does not require public documentation updates label Feb 27, 2023
@vstpme vstpme changed the title Return 100 latest alerts on alert group page Show 100 latest alerts on alert group page Feb 27, 2023
@Konstantinov-Innokentii
Copy link
Member

LGTM, left comment

@vstpme vstpme enabled auto-merge February 28, 2023 14:06
@vstpme vstpme added this pull request to the merge queue Feb 28, 2023
Merged via the queue into dev with commit a25fd42 Feb 28, 2023
@vstpme vstpme deleted the vadimkerr/fix-alert-order branch February 28, 2023 14:19
brojd pushed a commit that referenced this pull request Sep 18, 2024
# What this PR does
Make internal API return 100 latest alerts for alert group.

## Which issue(s) this PR fixes
#857

## Checklist

- [x] Tests updated
- [x] `CHANGELOG.md` updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants