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

Change POST to PUT and _security to security for API example #173453

Merged
merged 3 commits into from
Jan 3, 2024

Conversation

Terilia
Copy link
Member

@Terilia Terilia commented Dec 15, 2023

Summary

Just a small Doc Update. In the first example a POST was used and towards a non existing handle (_security instead of security)

In the basic license example posted below this one, this is already correct.

Checklist

All checked - none applied.
(Though first PR in this repository, I am sorry if any mistakes were made. If so - please point them out, will do better next time 😅 )

@Terilia Terilia requested a review from a team as a code owner December 15, 2023 11:50
Copy link
Contributor

Documentation preview:

@Terilia
Copy link
Member Author

Terilia commented Jan 3, 2024

@elasticmachine merge upstream

@Terilia Terilia added release_note:skip Skip the PR/issue when compiling release notes docs backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Jan 3, 2024
@Terilia Terilia merged commit 96ada94 into main Jan 3, 2024
13 checks passed
@Terilia Terilia deleted the Terilia-patch-2 branch January 3, 2024 14:07
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 3, 2024
…#173453)

## Summary

Just a small Doc Update. In the first example a POST was used and
towards a non existing handle (`_security` instead of `security`)

In the basic license example posted below this one, this is already
correct.

### Checklist

All checked - none applied.
(Though first PR in this repository, I am sorry if any mistakes were
made. If so - please point them out, will do better next time 😅 )

---------

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 96ada94)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.12

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 Jan 3, 2024
…173453) (#174179)

# Backport

This will backport the following commits from `main` to `8.12`:
- [Change POST to PUT and _security to security for API example
(#173453)](#173453)

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

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

<!--BACKPORT [{"author":{"name":"Alex
S","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-01-03T14:07:21Z","message":"Change
POST to PUT and _security to security for API example (#173453)\n\n##
Summary\r\n\r\nJust a small Doc Update. In the first example a POST was
used and\r\ntowards a non existing handle (`_security` instead of
`security`)\r\n\r\nIn the basic license example posted below this one,
this is already\r\ncorrect.\r\n\r\n### Checklist\r\n\r\nAll checked -
none applied. \r\n(Though first PR in this repository, I am sorry if any
mistakes were\r\nmade. If so - please point them out, will do better
next time 😅 )\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"96ada947238974a159781013519fe8398593aecc","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","docs","backport:prev-minor","v8.13.0"],"title":"Change
POST to PUT and _security to security for API
example","number":173453,"url":"https://github.com/elastic/kibana/pull/173453","mergeCommit":{"message":"Change
POST to PUT and _security to security for API example (#173453)\n\n##
Summary\r\n\r\nJust a small Doc Update. In the first example a POST was
used and\r\ntowards a non existing handle (`_security` instead of
`security`)\r\n\r\nIn the basic license example posted below this one,
this is already\r\ncorrect.\r\n\r\n### Checklist\r\n\r\nAll checked -
none applied. \r\n(Though first PR in this repository, I am sorry if any
mistakes were\r\nmade. If so - please point them out, will do better
next time 😅 )\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"96ada947238974a159781013519fe8398593aecc"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.13.0","branchLabelMappingKey":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/173453","number":173453,"mergeCommit":{"message":"Change
POST to PUT and _security to security for API example (#173453)\n\n##
Summary\r\n\r\nJust a small Doc Update. In the first example a POST was
used and\r\ntowards a non existing handle (`_security` instead of
`security`)\r\n\r\nIn the basic license example posted below this one,
this is already\r\ncorrect.\r\n\r\n### Checklist\r\n\r\nAll checked -
none applied. \r\n(Though first PR in this repository, I am sorry if any
mistakes were\r\nmade. If so - please point them out, will do better
next time 😅 )\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"96ada947238974a159781013519fe8398593aecc"}}]}]
BACKPORT-->

Co-authored-by: Alex S <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) docs release_note:skip Skip the PR/issue when compiling release notes v8.12.0 v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants