Skip to content

[8.x] [Response Ops][Rules] Add New Rule Form to Stack Management (#194655)#196290

Merged
cnasikas merged 1 commit intoelastic:8.xfrom
cnasikas:backport/8.x/pr-194655
Oct 15, 2024
Merged

[8.x] [Response Ops][Rules] Add New Rule Form to Stack Management (#194655)#196290
cnasikas merged 1 commit intoelastic:8.xfrom
cnasikas:backport/8.x/pr-194655

Conversation

@cnasikas
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…194655)

## Summary

Enables and adds the new rule form to stack management. We are only
going to turn this on for stack management for now until we are
confident that this is fairly bug free.

### To test:

1. Switch `USE_NEW_RULE_FORM_FEATURE_FLAG` to true
2. Navigate to stack management -> rules list
3. Click "Create rule"
4. Assert the user is navigated to the new form
5. Create rule
6. Assert the user is navigated to the rule details page
7. Click "Edit"
8. Edit rule
9. Assert the user is navigated to the rule details page
10. Try editing a rule in the rules list and assert everything works as
expected

We should also make sure this rule form is not enabled in other
solutions.

### Checklist
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Christos Nasikas <christos.nasikas@elastic.co>
(cherry picked from commit 5c2df63)

# Conflicts:
#	x-pack/test_serverless/functional/config.base.ts
@cnasikas cnasikas added the backport This PR is a backport of another PR label Oct 15, 2024
@cnasikas cnasikas enabled auto-merge (squash) October 15, 2024 12:07
@cnasikas cnasikas merged commit 99bddf8 into elastic:8.x Oct 15, 2024
@cnasikas cnasikas deleted the backport/8.x/pr-194655 branch October 15, 2024 13:57
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
triggersActionsUi 805 850 +45

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/alerts-ui-shared 305 304 -1
@kbn/rule-data-utils 127 133 +6
triggersActionsUi 567 568 +1
total +6

Async chunks

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

id before after diff
triggersActionsUi 1.6MB 1.7MB +81.9KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
triggersActionsUi 123.3KB 127.4KB +4.1KB
Unknown metric groups

API count

id before after diff
@kbn/alerts-ui-shared 321 320 -1
@kbn/rule-data-utils 130 136 +6
triggersActionsUi 593 594 +1
total +6

async chunk count

id before after diff
triggersActionsUi 64 66 +2

ESLint disabled line counts

id before after diff
@kbn/alerts-ui-shared 7 8 +1
triggersActionsUi 127 130 +3
total +4

Total ESLint disabled count

id before after diff
@kbn/alerts-ui-shared 8 9 +1
triggersActionsUi 133 136 +3
total +4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants