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

[Backport 2.x] Rule flyout opening from Findings and Alerts page #230

Merged
merged 1 commit into from
Dec 19, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 11e47b3 from #219

* rule flyout opening from findings and alerts page

Signed-off-by: Aleksandar Djindjic <[email protected]>

* update cypress test for findings page

Signed-off-by: Aleksandar Djindjic <[email protected]>

* make code shorter

Signed-off-by: Aleksandar Djindjic <[email protected]>

* toast error notifications for rule deletion

Signed-off-by: Aleksandar Djindjic <[email protected]>

* cleanup component state

Signed-off-by: Aleksandar Djindjic <[email protected]>

* avoid as any in favor of RuleItemInfoBase

Signed-off-by: Aleksandar Djindjic <[email protected]>

* fix cypress test for rules

Signed-off-by: Aleksandar Djindjic <[email protected]>

Signed-off-by: Aleksandar Djindjic <[email protected]>
(cherry picked from commit 11e47b3)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team December 16, 2022 18:25
@amsiglan amsiglan merged commit 8a2172d into 2.x Dec 19, 2022
@amsiglan amsiglan deleted the backport/backport-219-to-2.x branch December 21, 2022 18:29
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Feb 22, 2023
…#219) (opensearch-project#230)

* rule flyout opening from findings and alerts page

Signed-off-by: Aleksandar Djindjic <[email protected]>

* update cypress test for findings page

Signed-off-by: Aleksandar Djindjic <[email protected]>

* make code shorter

Signed-off-by: Aleksandar Djindjic <[email protected]>

* toast error notifications for rule deletion

Signed-off-by: Aleksandar Djindjic <[email protected]>

* cleanup component state

Signed-off-by: Aleksandar Djindjic <[email protected]>

* avoid as any in favor of RuleItemInfoBase

Signed-off-by: Aleksandar Djindjic <[email protected]>

* fix cypress test for rules

Signed-off-by: Aleksandar Djindjic <[email protected]>

Signed-off-by: Aleksandar Djindjic <[email protected]>
(cherry picked from commit 11e47b3)

Co-authored-by: Aleksandar Djindjic <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Oct 12, 2023
…#219) (opensearch-project#230)

* rule flyout opening from findings and alerts page

Signed-off-by: Aleksandar Djindjic <[email protected]>

* update cypress test for findings page

Signed-off-by: Aleksandar Djindjic <[email protected]>

* make code shorter

Signed-off-by: Aleksandar Djindjic <[email protected]>

* toast error notifications for rule deletion

Signed-off-by: Aleksandar Djindjic <[email protected]>

* cleanup component state

Signed-off-by: Aleksandar Djindjic <[email protected]>

* avoid as any in favor of RuleItemInfoBase

Signed-off-by: Aleksandar Djindjic <[email protected]>

* fix cypress test for rules

Signed-off-by: Aleksandar Djindjic <[email protected]>

Signed-off-by: Aleksandar Djindjic <[email protected]>
(cherry picked from commit 11e47b3)

Co-authored-by: Aleksandar Djindjic <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants