Skip to content

[Rules] Adding actions to the rules details action menu (Part 2)#219790

Merged
baileycash-elastic merged 15 commits intoelastic:mainfrom
baileycash-elastic:rules-199421-1
May 8, 2025
Merged

[Rules] Adding actions to the rules details action menu (Part 2)#219790
baileycash-elastic merged 15 commits intoelastic:mainfrom
baileycash-elastic:rules-199421-1

Conversation

@baileycash-elastic
Copy link
Contributor

@baileycash-elastic baileycash-elastic commented Apr 30, 2025

Continuing implementation of #199421

Summary

This PR:

  • moves snooze to a separate button similar to the mockup attached to the issue
  • adds "run rule" and "update API key"
Screenshot 2025-05-01 at 3 13 11 PM

baileycash-elastic added a commit that referenced this pull request May 1, 2025
…9635)

## Summary

Partially implements #199421

This PR adds snooze and enable/disable options to the action menu of the
rule details header action menu.

Note: A second PR with the other 2 required actions and a slight
redesign is [here](#219790)


![Screenshot 2025-04-29 at 4 37
45 PM](https://github.com/user-attachments/assets/e0ba273d-56f0-433f-b1b0-bb2842ed634b)
![Screenshot 2025-04-29 at 4 37
37 PM](https://github.com/user-attachments/assets/fa6d70e9-3582-412f-aaa9-e0fb8c219d10)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 1, 2025
…stic#219635)

## Summary

Partially implements elastic#199421

This PR adds snooze and enable/disable options to the action menu of the
rule details header action menu.

Note: A second PR with the other 2 required actions and a slight
redesign is [here](elastic#219790)

![Screenshot 2025-04-29 at 4 37
45 PM](https://github.com/user-attachments/assets/e0ba273d-56f0-433f-b1b0-bb2842ed634b)
![Screenshot 2025-04-29 at 4 37
37 PM](https://github.com/user-attachments/assets/fa6d70e9-3582-412f-aaa9-e0fb8c219d10)

(cherry picked from commit 1ee2f62)
@prodsecmachine
Copy link
Collaborator

prodsecmachine commented May 1, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@baileycash-elastic baileycash-elastic changed the title Rules 199421 Part 2 [Rules] Adding actions to the rules details action menu (Part 2) May 1, 2025
@baileycash-elastic baileycash-elastic marked this pull request as ready for review May 1, 2025 19:11
@baileycash-elastic baileycash-elastic requested a review from a team as a code owner May 1, 2025 19:11
@baileycash-elastic baileycash-elastic added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label May 1, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@baileycash-elastic baileycash-elastic added backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes v9.1.0 v8.19.0 labels May 1, 2025
Copy link
Contributor

@kdelemme kdelemme left a comment

Choose a reason for hiding this comment

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

Some comments about react hook query usage that i think we should fix

@baileycash-elastic baileycash-elastic requested a review from a team May 2, 2025 13:12
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observability 1280 1282 +2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observability 1.3MB 1.3MB +1.6KB

History

@kdelemme
Copy link
Contributor

kdelemme commented May 7, 2025

One question, why the bell icons are in different state by default?

Screenshot 2025-05-07 at 9 59 44 AM

…ges/rule_details/components/header_actions.tsx


Reversing the condition to make the icon consistent with the interactive area of the status panel - and because the snooze state can be updated without updating the actual mute state
Copy link
Contributor

@kdelemme kdelemme left a comment

Choose a reason for hiding this comment

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

LGTM and tested locally.

@baileycash-elastic baileycash-elastic merged commit 0129955 into elastic:main May 8, 2025
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/14897800049

@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 12, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @baileycash-elastic

2 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @baileycash-elastic

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @baileycash-elastic

kibanamachine added a commit that referenced this pull request May 14, 2025
…2) (#219790) (#220458)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Rules] Adding actions to the rules details action menu (Part 2)
(#219790)](#219790)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Bailey
Cash","email":"bailey.cash@elastic.co"},"sourceCommit":{"committedDate":"2025-05-08T03:14:12Z","message":"[Rules]
Adding actions to the rules details action menu (Part 2)
(#219790)","sha":"0129955c5d67087c1a74ca2bbf133e25866a36a7","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-management","backport:version","v9.1.0","v8.19.0"],"title":"[Rules]
Adding actions to the rules details action menu (Part
2)","number":219790,"url":"https://github.com/elastic/kibana/pull/219790","mergeCommit":{"message":"[Rules]
Adding actions to the rules details action menu (Part 2)
(#219790)","sha":"0129955c5d67087c1a74ca2bbf133e25866a36a7"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/219790","number":219790,"mergeCommit":{"message":"[Rules]
Adding actions to the rules details action menu (Part 2)
(#219790)","sha":"0129955c5d67087c1a74ca2bbf133e25866a36a7"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Bailey Cash <bailey.cash@elastic.co>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 14, 2025
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
…stic#219635)

## Summary

Partially implements elastic#199421

This PR adds snooze and enable/disable options to the action menu of the
rule details header action menu.

Note: A second PR with the other 2 required actions and a slight
redesign is [here](elastic#219790)


![Screenshot 2025-04-29 at 4 37
45 PM](https://github.com/user-attachments/assets/e0ba273d-56f0-433f-b1b0-bb2842ed634b)
![Screenshot 2025-04-29 at 4 37
37 PM](https://github.com/user-attachments/assets/fa6d70e9-3582-412f-aaa9-e0fb8c219d10)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants