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

[Response Ops] [Rule Form] Remove V1 Rule Form Flyout #209171

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

[Response Ops] [Rule Form] Remove V1 Rule Form Flyout #209171

wants to merge 9 commits into from
+49 −8,223

Conversation

Zacqary
Copy link
Contributor

@Zacqary Zacqary commented Jan 31, 2025 •
edited
Loading

Summary

Closes #195211

Removes all old rule form flyout code, which should no longer be referenced at all after #206685

Checklist

@Zacqary Zacqary added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v9.0.0 backport:version Backport to applied version labels labels Jan 31, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 31, 2025 •
edited
Loading

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!

@Zacqary Zacqary mentioned this pull request Jan 31, 2025
Open
@Zacqary Zacqary force-pushed the 195211-v1-flyout-remove branch from 768889a to ee3b438 Compare March 5, 2025 15:56
@Zacqary
Remove V1 Rule Form flyout
3e0313f 
# Conflicts:
#	x-pack/platform/plugins/shared/triggers_actions_ui/.storybook/decorator.tsx
#	x-pack/platform/plugins/shared/triggers_actions_ui/common/experimental_features.ts
#	x-pack/platform/plugins/shared/triggers_actions_ui/public/common/get_experimental_features.test.tsx
@Zacqary Zacqary force-pushed the 195211-v1-flyout-remove branch from ee3b438 to 3e0313f Compare March 5, 2025 15:58
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 5, 2025 •
edited
Loading

💔 Build Failed

Failed CI Steps

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v9.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[ResponseOps][Rules] Use the new rule form to the create rules flyout
3 participants
@Zacqary @elasticmachine @kibanamachine