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] Creates custom landing page for the Py client book #2187

Merged
merged 3 commits into from
Mar 30, 2023

Conversation

szabosteve
Copy link
Contributor

@szabosteve szabosteve commented Mar 29, 2023

Overview

This PR adds a new landing page to the Python client book so it will be more in line with the main docs landing page, the ES guide landing page, and the Kibana guide landing page.

The content can be easily modified based on feedback.

@szabosteve
Copy link
Contributor Author

@elasticmachine test this please

<h2>Documentation</h2>
</p>
<p>
The official Python client provides a low-level client for communicating with an Elasticsearch cluster.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we change the description here? I want to get away from the whole "low-level" versus "high-level" distinction, especially for Python. Maybe this manifests elsewhere too, need to fix that as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sethmlarson Sure! Do you have any preferred alternative?

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe can copy JS with: "The official Python client provides one-to-one mapping with Elasticsearch REST APIs."

@sethmlarson sethmlarson merged commit ac094c0 into main Mar 30, 2023
@sethmlarson sethmlarson deleted the szabosteve/custom-landing-page branch March 30, 2023 12:58
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.

None yet

2 participants