Commits on Dec 15, 2020

1. [Security Solution][Detections] Update read-only callouts from warnin… …

   …g to info so they persist when dismissed (elastic#84904)
   
   **Addresses:** elastic#76587
   
   ## Summary
   
   In this PR I'm doing basically 2 things:
   
   1. Making readonly callouts we have in Detections `primary` instead of `warning` and thus persistable in local storage (if a callout is dismissed, we remember it there)
   2. Creating a reusable implementation for that.
   
   TODO:
   
   - [x] Adjust all callouts used in Detections to be of type `primary`
   - [x] Implement the local storage logic (dumb)
   - [x] Implement the local storage logic (reusable)
   - [x] Add a new user role: "Reader" (read-only user)
   - [x] Add Cypress tests
   
   Out of scope:
   
   - Add unit tests. I'll probably address this in a follow-up PR. Do you think it's worth it or better to wait until the rework (see the next point)?
   - Rework callouts to standardise them across Detections, Cases and other parts of the Security app? See my comment below in this PR.
   
   ## Screen recordings
   
   Quick demo of how this implementation works:
   
   - [primary callouts](https://drive.google.com/file/d/1SYQd_ihKPvzlVUxELI8qNEqLBOkL18Gd/view?usp=sharing)
   - [warning, danger](https://drive.google.com/file/d/1lrAFPyXNjOYSiEsUXxY_fjXsvmyDcdWY/view?usp=sharing) (callout types here were manually adjusted)
   
   ## Additional notes
   
   Cypress tests are based on the work done in elastic#81866.
   
   ![](https://puu.sh/GXwOd/1c855cb03f.png)

   

   banderror committed Dec 15, 2020
   Configuration menu

   • View commit details
   • Copy full SHA for 68a99ca
   • Browse repository at this point

   Copy the full SHA

   68a99ca View commit details

   Browse the repository at this point in the history