[Observability] [Cases] Cases in the observability app

[stephmilovic]

Summary

Cases in the observability app. Four new views:

1. All Cases

/observability/cases

2. Create Case

/observability/cases/create

3. Case Details

/observability/cases/:detailName

4. Configure Case

/observability/cases/configure

There is still a flag on Observability/Cases in the sidebar menu:
xpack.observability.unsafe.alertingExperience.enabled: true

Internal ticket: https://github.com/elastic/security-team/issues/853

It's working! Still to do:

• fix missing breadcrumbs
• connect userPermissions for each page
• connect useFetchAlertData
• connect showAlertDetails (in security we open timeline)
• update values for ruleDetailsNavigation
• No longer pursuing alerts in this PR. making the above mentioned properties optional in Cases UI wherever used: useFetchAlertData, showAlertDetails, ruleDetailsNavigation
• test!

[stephmilovic]

This is unused, but leaving here for when alerting is enabled unless someone objects?

[legrego]

Is the plan to merge this into the cases-rbac-poc branch before it merges to master, or will you rebase this against master once cases-rbac-poc merges?

I ask because cases-rbac-poc is already an enormous PR, and adding another large PR on top of it will make reviewing that much more difficult, especially for those of us who aren't intimately familiar with this part of the codebase.

[stephmilovic]

Is the plan to merge this into the cases-rbac-poc branch before it merges to master, or will you rebase this against master once cases-rbac-poc merges?

I ask because cases-rbac-poc is already an enormous PR, and adding another large PR on top of it will make reviewing that much more difficult, especially for those of us who aren't intimately familiar with this part of the codebase.

@legrego definitely waiting for cases-rbac-poc and reopening against master. Hoping my reviewers can approve both this PR and the new one I'll need to open so we can get this code merged quickly

[smith]

Looks good overall but I found a few problems:

• When viewing an individual case, breadcrumb for "cases" isn't clickable. I think you might need to put an href property somewhere
• On the table, filtering by a tag using the filters always returns an empty list for me
• When viewing an individual case and deleting it, I'm taken back to the observability overview page instead of the list of cases
• Once I've deleted a case, if I go back to that URL (http://localhost:5701/kbn/app/observability/cases/cad584d0-bf03-11eb-a23c-5924635d118c), I get a blank page with a toast about an error fetching data. There should be some kind of not found page

[katrin-freihofner]

@stephmilovic the screenshots look good to me (I don't have any data to try it). Looking forward to trying it out when it's ready!

[stephmilovic]

@stephmilovic the screenshots look good to me (I don't have any data to try it). Looking forward to trying it out when it's ready!

no need for any index data! just go in and create a case, then you will have data ;)

[stephmilovic]

Looks good overall but I found a few problems:

• When viewing an individual case, breadcrumb for "cases" isn't clickable. I think you might need to put an href property somewhere
• On the table, filtering by a tag using the filters always returns an empty list for me
• When viewing an individual case and deleting it, I'm taken back to the observability overview page instead of the list of cases
• Once I've deleted a case, if I go back to that URL (http://localhost:5701/kbn/app/observability/cases/cad584d0-bf03-11eb-a23c-5924635d118c), I get a blank page with a toast about an error fetching data. There should be some kind of not found page

changes made! thank you!

[stephmilovic]

@elasticmachine merge upstream

[jonathan-buttner]

This PR was against the cases-rbac-poc branch which has been merged to master. I accidentally deleted the feature branch which inadvertently closed this PR. A new PR was opened against master here: #101487

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 1211f7f
• Storybooks Preview

History

• 💚 Build #129005 succeeded 28152b8
• 💔 Build #128915 failed 6a66a6c
• 💔 Build #128905 failed 8f2cb30
• 💔 Build #128748 failed 1edb767
• 💔 Build #128511 failed bf02730

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