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.

Sign up for GitHub

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

Jump to bottom

fix: indexing statuses filter key #90

Merged
merged 1 commit into from
Nov 8, 2023
Merged

fix: indexing statuses filter key #90

merged 1 commit into from
Nov 8, 2023

Conversation

hopeyen
Copy link
Collaborator

@hopeyen hopeyen commented Nov 8, 2023 •
edited
Loading

Fix filter key for monitor_deployment_status
For #89

@hopeyen hopeyen added size:small Small p2 Medium priority type:bug Something isn't working labels Nov 8, 2023
@hopeyen hopeyen self-assigned this Nov 8, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 8, 2023

Pull Request Test Coverage Report for Build 6801804303

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 50.144%
Totals Coverage Status
Change from base Build 6801270499: 0.0%
Covered Lines: 1392
Relevant Lines: 2776
💛 - Coveralls

Theodus
Theodus approved these changes Nov 8, 2023
View reviewed changes
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

LGTM

@hopeyen hopeyen merged commit f43900c into main Nov 8, 2023
5 checks passed
@hopeyen hopeyen deleted the hope/fix-indexing-status branch November 8, 2023 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Theodus Theodus Theodus approved these changes

Assignees

@hopeyen hopeyen

Labels
p2 Medium priority size:small Small type:bug Something isn't working
Projects
Indexer
Status: 🚗 Merged
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hopeyen @Theodus