Skip to content

New documentation for 8 #1215

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

Merged
merged 20 commits into from
Apr 14, 2022
Merged

New documentation for 8 #1215

merged 20 commits into from
Apr 14, 2022

Conversation

ezimuel
Copy link
Contributor

@ezimuel ezimuel commented Apr 12, 2022

This PR contains the new documentation for elasticsearch-php v8. I removed some pages and the API reference since I need to rewrite the generator for that.

Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

I left a couple of comments, mostly tiny things. The most important one is the redirect.asciidoc file; it would be essential to keep that file in the repo to support our linking strategy. The docs-ci on the PR fails partially because this file would be removed in this PR. If you could add it back, I'd add the most important pages deleted in this PR to it.

ezimuel and others added 16 commits April 14, 2022 11:20
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
Co-authored-by: István Zoltán Szabó <[email protected]>
@ezimuel
Copy link
Contributor Author

ezimuel commented Apr 14, 2022

@szabosteve I added the redirects.asciidoc file with the changes that you suggested. Thanks for the review!

@szabosteve
Copy link
Contributor

@elasticmachine test this please

Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

LGTM!

@ezimuel ezimuel merged commit 145373b into elastic:master Apr 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants