-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Security Solution] Bulk action confirmation dialog should not mention "custom" rules #203149
Comments
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
@dplumlee While this one is low-impact, it looks a bit embarrassing and I think we should fix it soon. |
@nikitaindik I assigned this one to you since it's an UI bug and you should have capacity to handle it. |
…lastic#204313) **Resolves: elastic#203149 ## Summary This PR updates the bulk action modal dialogs by removing the term "custom." With prebuilt rules now being editable, the use of the word "custom" is no longer accurate or appropriate. ## Screenshots **Before** <img width="805" alt="Schermafbeelding 2024-12-14 om 12 09 58" src="https://github.com/user-attachments/assets/b354ee44-d565-486f-94d6-6851e9d6a7b2" /> **After** <img width="728" alt="Schermafbeelding 2024-12-14 om 11 56 04" src="https://github.com/user-attachments/assets/c9de12c3-8aca-4ee2-828c-71b05f3b451a" /> Work started on 14-Dec-2024 (cherry picked from commit 65bc6bc)
…lastic#204313) **Resolves: elastic#203149 ## Summary This PR updates the bulk action modal dialogs by removing the term "custom." With prebuilt rules now being editable, the use of the word "custom" is no longer accurate or appropriate. ## Screenshots **Before** <img width="805" alt="Schermafbeelding 2024-12-14 om 12 09 58" src="https://github.com/user-attachments/assets/b354ee44-d565-486f-94d6-6851e9d6a7b2" /> **After** <img width="728" alt="Schermafbeelding 2024-12-14 om 11 56 04" src="https://github.com/user-attachments/assets/c9de12c3-8aca-4ee2-828c-71b05f3b451a" /> Work started on 14-Dec-2024
…lastic#204313) **Resolves: elastic#203149 ## Summary This PR updates the bulk action modal dialogs by removing the term "custom." With prebuilt rules now being editable, the use of the word "custom" is no longer accurate or appropriate. ## Screenshots **Before** <img width="805" alt="Schermafbeelding 2024-12-14 om 12 09 58" src="https://github.com/user-attachments/assets/b354ee44-d565-486f-94d6-6851e9d6a7b2" /> **After** <img width="728" alt="Schermafbeelding 2024-12-14 om 11 56 04" src="https://github.com/user-attachments/assets/c9de12c3-8aca-4ee2-828c-71b05f3b451a" /> Work started on 14-Dec-2024
…lastic#204313) **Resolves: elastic#203149 ## Summary This PR updates the bulk action modal dialogs by removing the term "custom." With prebuilt rules now being editable, the use of the word "custom" is no longer accurate or appropriate. ## Screenshots **Before** <img width="805" alt="Schermafbeelding 2024-12-14 om 12 09 58" src="https://github.com/user-attachments/assets/b354ee44-d565-486f-94d6-6851e9d6a7b2" /> **After** <img width="728" alt="Schermafbeelding 2024-12-14 om 11 56 04" src="https://github.com/user-attachments/assets/c9de12c3-8aca-4ee2-828c-71b05f3b451a" /> Work started on 14-Dec-2024
…lastic#204313) **Resolves: elastic#203149 ## Summary This PR updates the bulk action modal dialogs by removing the term "custom." With prebuilt rules now being editable, the use of the word "custom" is no longer accurate or appropriate. ## Screenshots **Before** <img width="805" alt="Schermafbeelding 2024-12-14 om 12 09 58" src="https://github.com/user-attachments/assets/b354ee44-d565-486f-94d6-6851e9d6a7b2" /> **After** <img width="728" alt="Schermafbeelding 2024-12-14 om 11 56 04" src="https://github.com/user-attachments/assets/c9de12c3-8aca-4ee2-828c-71b05f3b451a" /> Work started on 14-Dec-2024
PR: #204313
Summary
Rules are mistakenly labeled as custom even when they are prebuilt.
Steps to Reproduce
Actual Result
A modal dialog appears stating that the action will only be applied to "X custom rules," even though no custom rules are installed.
Expected Result
The dialog should not mention custom rules. Instead, it should correctly identify that the selected rules are prebuilt.
The text was updated successfully, but these errors were encountered: