-
Notifications
You must be signed in to change notification settings - Fork 25k
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 simulate ref #58579
[DOCS] Add simulate ref #58579
Conversation
49d256b
to
43c2e56
Compare
25d868d
to
658d1ff
Compare
@elasticmachine retest this please |
e642ad6
to
0c2f836
Compare
Pinging @elastic/es-core-features (:Core/Features/Indices APIs) |
Pinging @elastic/es-docs (>docs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall.
I mostly left non-blocking nits and suggestions. However, the current changes include a cause
query param that didn't work in my testing.
Once that param is corrected or removed, I think this is good to go. Thanks @debadair.
Co-authored-by: James Rodewig <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
d7f2c16
to
a1b5a94
Compare
* [DOCS] Add simulate ref pages * Add links & experimental tags * Fixed simulate index response * Apply suggestions from code review Co-authored-by: James Rodewig <[email protected]> *Incorporate review feedback.
* [DOCS] Add simulate ref pages * Add links & experimental tags * Fixed simulate index response * Apply suggestions from code review Co-authored-by: James Rodewig <[email protected]> *Incorporate review feedback.
Adds ref pages for the template simulation APIs.
Preview:
Simulate index API
Simulate index template API