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.

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

[DOCS][OpenAPI] Add descriptions, examples, tags, responses for space APIs #195333

Merged
merged 35 commits into from
Oct 18, 2024

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Oct 7, 2024

Summary

Relates to #192153

This PR copies descriptions and response codes from the old spaces API docs into the automated OpenAPI document. It also adds some missing operation tags and summaries. Since per #188926 examples aren't integrated into the automated documents yet, I've added them via overlays.

It also removes the old API docs and creates temporary redirects until links can be cleaned out of the entire library. The multi-step example from the update saved object spaces API page is moved to the "managing saved objects" page. Preview: https://kibana_bk_195333.docs-preview.app.elstc.co/guide/en/kibana/master/managing-saved-objects.html#managing-saved-objects-share-to-space

@lcawl lcawl added Feature:Security/Spaces Platform Security - Spaces feature release_note:skip Skip the PR/issue when compiling release notes docs v8.16.0 backport:version Backport to applied version labels labels Oct 7, 2024
@lcawl lcawl changed the title [DOCS][OpenAPI] Add descriptions and examples for create, update space APIs [DOCS][OpenAPI] Add descriptions, examples, tags, responses for space APIs Oct 9, 2024
@lcawl lcawl marked this pull request as ready for review October 9, 2024 00:09
@lcawl lcawl requested a review from a team as a code owner October 9, 2024 00:09
@lcawl lcawl requested a review from a team October 9, 2024 00:10
@lcawl lcawl removed the request for review from a team October 9, 2024 00:44
@lcawl lcawl marked this pull request as draft October 11, 2024 02:32
@lcawl lcawl marked this pull request as ready for review October 11, 2024 18:30
@lcawl lcawl marked this pull request as draft October 15, 2024 20:32
@lcawl
Copy link
Contributor Author

lcawl commented Oct 16, 2024

Another thing I've noticed that we are losing cross links fro documentation once we move it into the OAS description...Not sure if we can do anything about it, but this has a an impact from user perspective when navigation through our docs.

Generally OpenAPI documents support markdown per https://spec.openapis.org/oas/v3.0.3#rich-text-formatting. I haven't tried adding links within the automatically-generated content yet, in part because things are still moving around enough that I think link breakages would occur a lot right now and we don't have any sort of link checking set up yet. In the longer term, particularly when it comes to linking from API docs to our other guides, I'm thinking externalDocs would potentially give us more options for keeping those URLs up-to-date and version-specific.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

Copy link
Contributor

@elena-shostak elena-shostak left a comment

Choose a reason for hiding this comment

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

LGTM from my side!

@lcawl lcawl added the v8.17.0 label Oct 18, 2024
@lcawl lcawl enabled auto-merge (squash) October 18, 2024 17:15
@lcawl lcawl merged commit 2656e5a into elastic:main Oct 18, 2024
23 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.x

https://github.com/elastic/kibana/actions/runs/11409485180

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 18, 2024
… APIs (elastic#195333)

Co-authored-by: Elena Shostak <[email protected]>
(cherry picked from commit 2656e5a)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 18, 2024
… APIs (elastic#195333)

Co-authored-by: Elena Shostak <[email protected]>
(cherry picked from commit 2656e5a)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.16
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 18, 2024
…r space APIs (#195333) (#196943)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[DOCS][OpenAPI] Add descriptions, examples, tags, responses for space
APIs (#195333)](#195333)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Lisa
Cawley","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-18T19:03:06Z","message":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space APIs
(#195333)\n\nCo-authored-by: Elena Shostak
<[email protected]>","sha":"2656e5a4ff79277c9a370ee24d6692ccb1d3142c","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Security/Spaces","release_note:skip","v9.0.0","docs","v8.16.0","backport:version","v8.17.0"],"title":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space
APIs","number":195333,"url":"https://github.com/elastic/kibana/pull/195333","mergeCommit":{"message":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space APIs
(#195333)\n\nCo-authored-by: Elena Shostak
<[email protected]>","sha":"2656e5a4ff79277c9a370ee24d6692ccb1d3142c"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195333","number":195333,"mergeCommit":{"message":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space APIs
(#195333)\n\nCo-authored-by: Elena Shostak
<[email protected]>","sha":"2656e5a4ff79277c9a370ee24d6692ccb1d3142c"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Lisa Cawley <[email protected]>
kibanamachine added a commit that referenced this pull request Oct 18, 2024
… space APIs (#195333) (#196944)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[DOCS][OpenAPI] Add descriptions, examples, tags, responses for space
APIs (#195333)](#195333)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Lisa
Cawley","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-18T19:03:06Z","message":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space APIs
(#195333)\n\nCo-authored-by: Elena Shostak
<[email protected]>","sha":"2656e5a4ff79277c9a370ee24d6692ccb1d3142c","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Security/Spaces","release_note:skip","v9.0.0","docs","v8.16.0","backport:version","v8.17.0"],"title":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space
APIs","number":195333,"url":"https://github.com/elastic/kibana/pull/195333","mergeCommit":{"message":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space APIs
(#195333)\n\nCo-authored-by: Elena Shostak
<[email protected]>","sha":"2656e5a4ff79277c9a370ee24d6692ccb1d3142c"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195333","number":195333,"mergeCommit":{"message":"[DOCS][OpenAPI]
Add descriptions, examples, tags, responses for space APIs
(#195333)\n\nCo-authored-by: Elena Shostak
<[email protected]>","sha":"2656e5a4ff79277c9a370ee24d6692ccb1d3142c"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Lisa Cawley <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels docs Feature:Security/Spaces Platform Security - Spaces feature release_note:skip Skip the PR/issue when compiling release notes v8.16.0 v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants