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] Add space_id path parameter to saved_object APIs #47627

Merged
merged 4 commits into from
Oct 18, 2019

Conversation

mellieA
Copy link

@mellieA mellieA commented Oct 8, 2019

Summary

Updated saved_object APIs with space_id

closes #38788
[skip ci]

@mellieA mellieA added Team:Docs v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.4.0 v7.5.0 labels Oct 8, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

==== Path parameters

`space_id`::
(Optional, string) An identifier for the space. If `space_id` is not provided in the URL the default space is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comma in between URL and the

[[saved-objects-api-bulk-get-request-body]]
==== Request Body

`type`::
(Required, string) Valid options include `visualization`, `dashboard`, `search`, `index-pattern`, `config`, and `timelion-sheet`.

`id`::
(Required, string) ID of the retrieved object.
(Required, string) ID of the retrieved object. This can be a Kibana's unique identifier or a custom one.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would change the second sentence to:

The ID includes the {kib} unique identifier or a custom identifier.

(array) Top-level property the contains objects that represent the response for each of the requested objects. The order of the objects in the response is identical to the order of the objects in the request.

Saved objects that are unable to persist are replaced with an error object.

[[saved-objects-api-bulk-get-body-codes]]
==== Response code

`200`::
`200`::
Indicates a successfully call.
Copy link
Contributor

Choose a reason for hiding this comment

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

successfully should be successful

[[saved-objects-api-create-path-params]]
==== Path parameters

`space_id`::
(Optional, string) An identifier for the space. If `space_id` is not provided in the URL the default space is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comma in between URL and the

[[saved-objects-api-delete-path-params]]
==== Path parameters

`space_id`::
(Optional, string) An identifier for the space. If `space_id` is not provided in the URL the default space is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comma in between URL and the

@@ -50,3 +56,60 @@ The API returns the following:
}
}
--------------------------------------------------

The following example retrieves a dashboard object in the `testspace` by id.
Copy link
Contributor

Choose a reason for hiding this comment

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

id should be ID

--------------------------------------------------
// KIBANA

A successful call returns a response code of `200` and a response body
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove A successful call returns a response code of 200 and a response body containing a JSON structure similar to the following example:, then replace with:

The API returns the following:

==== Path parameters

`space_id`::
(Optional, string) An identifier for the space. If `space_id` is not provided in the URL the default space is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comma in between URL and the

==== Path parameters

`space_id`::
(Optional, string) An identifier for the space. If `space_id` is not provided in the URL the default space is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comma in between URL and the

==== Path parameters

`space_id`::
(Optional, string) An identifier for the space. If `space_id` is not provided in the URL the default space is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comma in between URL and the

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!

@mellieA mellieA merged commit 8d34f25 into elastic:master Oct 18, 2019
@thomasneirynck
Copy link
Contributor

thomasneirynck commented Mar 23, 2020

hi @KOTungseth , @mellieA, does this need to be backported to the older 7.x branches?

See this discuss question: https://discuss.elastic.co/t/space-id-parameter-for-the-savedobjects-api/224548

mellieA pushed a commit to mellieA/kibana that referenced this pull request Mar 23, 2020
* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for elastic#47627

* [DOCS] Apply suggested edits for elastic#47627 to delete API
mellieA pushed a commit to mellieA/kibana that referenced this pull request Mar 23, 2020
* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for elastic#47627

* [DOCS] Apply suggested edits for elastic#47627 to delete API
mellieA pushed a commit to mellieA/kibana that referenced this pull request Mar 23, 2020
* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for elastic#47627

* [DOCS] Apply suggested edits for elastic#47627 to delete API
mellieA pushed a commit that referenced this pull request Mar 23, 2020
)

* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for #47627

* [DOCS] Apply suggested edits for #47627 to delete API
mellieA pushed a commit that referenced this pull request Mar 23, 2020
)

* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for #47627

* [DOCS] Apply suggested edits for #47627 to delete API
mellieA pushed a commit that referenced this pull request Mar 23, 2020
)

* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for #47627

* [DOCS] Apply suggested edits for #47627 to delete API
mellieA pushed a commit that referenced this pull request Mar 23, 2020
)

* [DOCS] Add space_id path parameter to saved_object APIs

* [DOCS] Apply suggested edits for #47627

* [DOCS] Apply suggested edits for #47627 to delete API

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Docs v7.4.0 v7.5.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update API documentation to include effects of space prefix in URL
4 participants