Skip to content
This repository was archived by the owner on Jul 8, 2025. It is now read-only.

fix: alerts line chart sorting #63

Merged
merged 1 commit into from
Jan 14, 2025
Merged

fix: alerts line chart sorting #63

merged 1 commit into from
Jan 14, 2025

Conversation

peppescg
Copy link
Collaborator

BEFORE
Screenshot 2025-01-14 at 19 38 36
AFTER
Screenshot 2025-01-14 at 19 37 27

@peppescg peppescg requested a review from danbarr January 14, 2025 18:44
@peppescg peppescg self-assigned this Jan 14, 2025
@peppescg peppescg linked an issue Jan 14, 2025 that may be closed by this pull request
Alerts by date graph is out of order #48
Closed
@peppescg peppescg enabled auto-merge (squash) January 14, 2025 18:44
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 12774312027

Details

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

danbarr
danbarr approved these changes Jan 14, 2025
View reviewed changes
Copy link
Collaborator

@danbarr danbarr left a comment

Choose a reason for hiding this comment

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

Tested locally with my DB, looks great!

@peppescg peppescg merged commit ce8146c into main Jan 14, 2025
7 checks passed
@peppescg peppescg deleted the issues/48 branch January 14, 2025 18:51
@stacklokbot stacklokbot mentioned this pull request Jan 15, 2025
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@danbarr danbarr danbarr approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@peppescg peppescg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Alerts by date graph is out of order
3 participants
@peppescg @coveralls @danbarr