[Security Solution][Endpoint][Host Isolation] Hide isolate host option in alert details rather than disabling#111064
Conversation
|
Pinging @elastic/esecurity-onboarding-and-lifecycle-mgt (Feature:Endpoint) |
|
Pinging @elastic/security-onboarding-and-lifecycle-mgt (Team:Onboarding and Lifecycle Mgt) |
| isHostIsolationPanelOpen === false | ||
| isHostIsolationPanelOpen === false && | ||
| loadingHostIsolationStatus === false && | ||
| agentStatus !== HostStatus.UNENROLLED |
There was a problem hiding this comment.
need to confirm if we should still have the isolate option disabled when the host status is UNENROLLED
|
The menu item showing up suddenly doesn't seem right. Makes the menu jump in size. Is there no way to keep the item disabled until the API response is done? |
That is actually what was originally happening, but I think because the API calls take a variable amount of time to become active, it might confuse the user more than if we were just to hide the option until it is valid. |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / general / Chrome X-Pack UI Functional Tests Basic License.x-pack/test/functional_basic/apps/ml/permissions/read_ml_access·ts.apps machine learning basic license permissions for user with read ML access (ft_ml_viewer) should display elements on Index Data Visualizer page correctlyStandard OutStack TraceMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
…n in alert details rather than disabling (elastic#111064)
…n in alert details rather than disabling (elastic#111064)
…-link-to-kibana-app * 'master' of github.com:elastic/kibana: (61 commits) [Logs UI] Fix alert previews for thresholds of `0` (elastic#111150) [Archive Migration][Partial] discover apps-discover (elastic#110437) [APM] Set start date of APM ML job to -4 weeks (elastic#111375) [ML] APM Latency Correlations: Code consolidation. (elastic#110790) [Discover] Fix indices permission for multiline test (elastic#111284) [Detection Rules] Add 7.15 rules (elastic#111464) [Security Solution][Endpoint][Host Isolation] Hide isolate host option in alert details rather than disabling (elastic#111064) React version of angular license view (elastic#111317) [APM] Fix link in readme (elastic#111362) [Security Solution] add agent field to generator (elastic#111428) [Dashboard] Retain Tags on Quicksave (elastic#111015) Reorder App Search ingestion methods (elastic#111361) Port performance docs to new docs system. (elastic#111063) [Security Solution][RAC] Fixes updatedAt loading bug (elastic#111010) [sample data] update web log geo.src field to match country code of geo.coordinates (elastic#110885) [Security solution] [Endpoint] Fix bad artifact migration (elastic#111294) Fix copy typo. (elastic#111203) [build] Remove empty optimize directory (elastic#111393) [Maps] fix term join not updating when editing right field (elastic#111030) [Fleet] Set default settings in component template instead of the index template (elastic#111197) ... # Conflicts: # x-pack/plugins/reporting/public/management/__snapshots__/report_listing.test.tsx.snap # x-pack/plugins/reporting/public/management/report_listing.test.tsx
…n in alert details rather than disabling (elastic#111064)
Summary
Screenshot
Using slow 3g network connection
