Skip to content

[7.x] [Security Solution] [Detections] Disable edit button when user does not have actions privileges w/ rule + actions (#80220)#81055

Merged
dhurley14 merged 1 commit intoelastic:7.xfrom
dhurley14:backport/7.x/pr-80220
Oct 20, 2020
Merged

[7.x] [Security Solution] [Detections] Disable edit button when user does not have actions privileges w/ rule + actions (#80220)#81055
dhurley14 merged 1 commit intoelastic:7.xfrom
dhurley14:backport/7.x/pr-80220

Conversation

@dhurley14
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…ot have actions privileges w/ rule + actions (elastic#80220)

* disable edit button only when there is an action present on the rule to be edited, but the user attempting the edit does not have actions privileges

* adds tooltip to explain why the edit rule button is disabled

* prevent user from editing rules with actions on the all rules table

* adds tooltip to appear on all rules table

* updates tests for missing params and missing mock of useKibana

* disable activate switch on all rules table and rule details page

* remove as casting in favor of a boolean type guard to ensure actions.show capabilities are a boolean even though tye are typed as a boolean | Record

* disable duplicate rule functionality for rules with actions

* fix positioning of tooltips and add tooltip to rule duplicate button in overflow button

* update tests

* WIP - display bulk actions dropdown options as disabled + add tooltips describing why they are disabled

* add eui tool tip as child of of each context menu item

* PR feedback and utilize map of rule ids to rules to replace usage of array.finds

* update snapshot

* fix mocks

* fix mocks

* update wording with feedback from design team

Co-authored-by: Patryk Kopycinski <contact@patrykkopycinski.com>
@dhurley14 dhurley14 added the backport This PR is a backport of another PR label Oct 19, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
securitySolution 2057 2058 +1

async chunks size

id before after diff
securitySolution 8.1MB 8.1MB +6.1KB

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

@dhurley14 dhurley14 merged commit 075efe4 into elastic:7.x Oct 20, 2020
@dhurley14 dhurley14 deleted the backport/7.x/pr-80220 branch October 20, 2020 13:42
@MindyRS MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Oct 27, 2020
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 Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants