-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[DOCS] Continued: Clarify upgrades and edit generally #3161
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
base: main
Are you sure you want to change the base?
Conversation
🔍 Preview links for changed docs |
|
@eedugon this isn't final yet, just hoping it's a step in the right direction. LMKWYT! |
This reverts commit fb7be17.
|
pausing this; will continue after #3194 is merged |
| In the Python client, compatibility mode is always enabled. | ||
| **What to expect in compatibility mode:** | ||
|
|
||
| If you're using the Python client in compatibility mode to work with the next major version of {{es}}: |
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.
| If you're using the Python client in compatibility mode to work with the next major version of {{es}}: | |
| If you're taking advantage of the Python client compatibility mode to work with the next major version of {{es}}: |
I think the wording should reflect that this is not something that can be turned on or off.
|
|
||
| If you're using the Python client in compatibility mode to work with the next major version of {{es}}: | ||
|
|
||
| * API calls affected by [breaking changes](../release-notes/breaking-changes.md) might fail or behave differently. |
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.
This is from a previous PR, but @pquentin noted that the breaking changes here refer to the server's next major release. This link should point to the server's breaking changes page instead of the client's.
Some quick improvements as part of https://github.com/elastic/docs-content-internal/issues/421.
Continues work started in #3138
AI-Assisted: This PR was assisted by Cursor.
Usage: Brainstorming; finding patterns in our docs; general ES research