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

Alert history ordered, add request arg and move basic info at top #558

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Alert history ordered, add request arg and move basic info at top #558

merged 1 commit into from
Sep 25, 2024

Conversation

thinkst-sharukh
Copy link
Contributor

@thinkst-sharukh thinkst-sharukh commented Aug 2, 2024 •
edited
Loading

Proposed changes

This pull request includes the following updates and improvements:

  1. Alert History Ordered: The alert history is now ordered in descending order for better chronological visibility.
  2. Add Request Argument: The missing request argument has been added to the basic information section, enhancing its completeness and accuracy.
  3. Move Basic Info to the Top: The basic information has been repositioned to the top of the section, improving visibility and user experience.

image

@thinkst-sharukh thinkst-sharukh force-pushed the alert_history_hits_order_and_request_arg branch from 9c4f1fb to 9b5ecc1 Compare September 9, 2024 06:18
@thinkst-sharukh thinkst-sharukh force-pushed the alert_history_hits_order_and_request_arg branch from 9b5ecc1 to 5831caf Compare September 25, 2024 05:08
@thinkst-sharukh thinkst-sharukh merged commit 85b0c9f into master Sep 25, 2024
@thinkst-sharukh thinkst-sharukh deleted the alert_history_hits_order_and_request_arg branch September 25, 2024 13:10
@thinkst-sharukh thinkst-sharukh restored the alert_history_hits_order_and_request_arg branch September 25, 2024 13:10
@thinkst-sharukh thinkst-sharukh deleted the alert_history_hits_order_and_request_arg branch September 26, 2024 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wleightond wleightond wleightond approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thinkst-sharukh @wleightond