Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DOCS] Add find case APIs #127686

Merged
merged 7 commits into from
Mar 22, 2022
Merged

[DOCS] Add find case APIs #127686

merged 7 commits into from
Mar 22, 2022

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Mar 15, 2022 •
edited
Loading

Summary

Relates to #126956

This PR copies the "find" subset of cases APIs from the Security Guide to the Kibana Guide. It also formats the content to align with existing Kibana APIs and updates the owner parameters to indicate support for all types of cases.

This PR must be merged after elastic/docs#2395 and #127647

Preview

@lcawl lcawl mentioned this pull request Mar 15, 2022
Closed
31 tasks
@lcawl lcawl added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting docs Feature:Cases Cases feature v8.2.0 labels Mar 15, 2022
@lcawl lcawl marked this pull request as ready for review March 15, 2022 23:01
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops-cases (Feature:Cases)

@lcawl lcawl requested review from KOTungseth and a team March 15, 2022 23:26
KOTungseth
KOTungseth approved these changes Mar 16, 2022
View reviewed changes
Copy link
Contributor

@KOTungseth KOTungseth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cnasikas cnasikas mentioned this pull request Mar 18, 2022
Merged
cnasikas
cnasikas approved these changes Mar 22, 2022
View reviewed changes
Copy link
Member

@cnasikas cnasikas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for doing this! LGTM!

peteharverson
peteharverson approved these changes Mar 22, 2022
View reviewed changes
Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest changes LGTM

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

History

  • 💚 Build #32429 succeeded 39609bae03faa53aea1b75f13578ac9617b53844
  • 💚 Build #32266 succeeded c85cb034ec7983ec40fdddcbfe1e97d69a1006aa
  • 💚 Build #31874 succeeded ed7b3331c93678e255350b47d6e0e31398421e23
  • 💚 Build #31663 succeeded c0649eebb4a6733ac446d22f5b2849ec4d9f6189
  • 💚 Build #31662 succeeded 1f36fd9da172bf319fa6acd1b31a11cf46624d9a
  • 💚 Build #30764 succeeded 39ca0e22ca8d0b64cdd5ed4df20dc36116ec6943

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

@lcawl lcawl merged commit 130823a into elastic:main Mar 22, 2022
@lcawl lcawl deleted the find-case-apis branch March 22, 2022 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peteharverson peteharverson peteharverson approved these changes

@nastasha-solomon nastasha-solomon nastasha-solomon left review comments

@cnasikas cnasikas cnasikas approved these changes

@KOTungseth KOTungseth KOTungseth approved these changes

Assignees
No one assigned
Labels
backport:skip This commit does not require backporting docs Feature:Cases Cases feature release_note:skip Skip the PR/issue when compiling release notes v8.2.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@lcawl @elasticmachine @kibana-ci @peteharverson @cnasikas @KOTungseth @nastasha-solomon