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

fix example data in docs #166

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

loru88
Copy link
Contributor

@loru88 loru88 commented Mar 6, 2024

Description

  • some examples were based on legacy Elasticsearch version
  • switch to string instead of number to match the OpenSearch type
  • add deprecation message to opensearch_ism_policy_mapping

Issues Resolved

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

- some where based on legacy Elasticsearch version
- switch to string instead of number to match the OpenSearch type
- add deprecation message to opensearch_ism_policy_mapping

Signed-off-by: Lorenzo <[email protected]>
@loru88
Copy link
Contributor Author

loru88 commented Mar 6, 2024

partial duplicate of #165

Also it shows the same deprecation message in the documentation merged in #159

@loru88 loru88 changed the title fix example data fix example data in docs Mar 6, 2024
@prudhvigodithi
Copy link
Collaborator

Thanks @loru88 LGTM, @rblcoder can you please add your review as well?

@rblcoder
Copy link
Collaborator

@loru88 Few of the changes remain to be replicated in the tests as well. They are minor ones. Can you please change the corresponding tests as well?

@loru88 loru88 force-pushed the fix_index_template_doc branch 3 times, most recently from 9d32f1a to bdb5b5f Compare March 13, 2024 07:16
@loru88
Copy link
Contributor Author

loru88 commented Mar 13, 2024

@loru88 Few of the changes remain to be replicated in the tests as well. They are minor ones. Can you please change the corresponding tests as well?

Thank you, I just fixed the issues. I updated the .go. files and I also fixed the OpenSearch spelling because it was only fixed in the generated file but it got reverted every time I run the command tfplugindocs.

Copy link
Collaborator

@rblcoder rblcoder left a comment

Choose a reason for hiding this comment

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

Thank you @loru88 , the updated examples are working.

@loru88
Copy link
Contributor Author

loru88 commented Mar 13, 2024

that's great, thank you @rblcoder . Can you approve the Workflow run, please?

@prudhvigodithi prudhvigodithi merged commit 92ce85a into opensearch-project:main Mar 14, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants