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.

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

[Cloud Posture] Rule flyout updates #132327

Merged
merged 10 commits into from
May 18, 2022
Merged

Conversation

JordanSh
Copy link
Contributor

@JordanSh JordanSh commented May 17, 2022

Summary

  • Rule flyout now uses shared lists from the Rule and Overview -> Remediation sections in the findings flyout.
    So basically every change we do in the findings flyout at those sections, we automatically get it in the rule flyout as well.
    checked with @tinnytintin10 and he liked it :)
  • Section column was renamed to CIS Section and now displays data
  • Removed rego tab

image

image

image

@JordanSh JordanSh added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:Cloud Security Cloud Security team related v8.3.0 labels May 17, 2022
@JordanSh JordanSh self-assigned this May 17, 2022
@JordanSh JordanSh changed the title Rule flyout updates [Cloud Posture] Rule flyout updates May 17, 2022
@JordanSh JordanSh force-pushed the rule-flyout-updates branch from 64f8c60 to 931f0ef Compare May 17, 2022 13:44
@JordanSh JordanSh marked this pull request as ready for review May 17, 2022 13:50
@JordanSh JordanSh requested a review from a team as a code owner May 17, 2022 13:50
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security Posture)

Copy link

@tinnytintin10 tinnytintin10 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Contributor

@orouz orouz left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -152,7 +91,7 @@ const RuleOverviewTab = ({ rule, toggleRule }: { rule: RuleSavedObject; toggleRu
</span>
</EuiFlexItem>
<EuiFlexItem>
<EuiDescriptionList compressed={false} listItems={getOverviewCard(rule)} />
<EuiDescriptionList listItems={getRuleList(rule.attributes)} />
Copy link
Contributor

Choose a reason for hiding this comment

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

is there a difference now that this one doesn't have compressed=false ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no, false is the default

@JordanSh JordanSh force-pushed the rule-flyout-updates branch from 877843e to 9f62773 Compare May 17, 2022 14:24
@JordanSh JordanSh enabled auto-merge (squash) May 17, 2022 14:25
@JordanSh JordanSh force-pushed the rule-flyout-updates branch 2 times, most recently from cd0fc4d to 7af8dc8 Compare May 18, 2022 08:34
@JordanSh JordanSh force-pushed the rule-flyout-updates branch from 7af8dc8 to a10e7fd Compare May 18, 2022 09:22
@JordanSh JordanSh merged commit b009d50 into elastic:main May 18, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
cloudSecurityPosture 182.6KB 181.1KB -1.4KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @JordanSh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants