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

Help aws users find the configuration information they need #482

Merged
merged 3 commits into from
May 8, 2017

Conversation

rromanchuk
Copy link
Contributor

Related:
#301

#302

Add an example how to successfully use chewy with amazon ElasticSearch using an IAM policy.
@pyromaniac
Copy link
Contributor

Hey man! Sorry I missed this somehow. Can you please resolve the conflict so I could merge it?

@pyromaniac
Copy link
Contributor

Never mind, I did it, thanks for this contribution!

@pyromaniac pyromaniac merged commit ef35cec into toptal:master May 8, 2017
@@ -15,6 +15,7 @@ Chewy is an ODM and wrapper for [the official Elasticsearch client](https://gith
* [Why Chewy?](#why-chewy)
* [Usage](#usage)
* [Client settings](#client-settings)
* [AWS ElasticSearch configuration] (#aws-elastic-search)

Choose a reason for hiding this comment

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

Hey @pyromaniac, there's an extra white-space here, which is messing up the Markdown styling.
It should be:
* [AWS ElasticSearch configuration](#aws-elastic-search)

Copy link
Contributor

Choose a reason for hiding this comment

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

Will fix it in another PR, thanks.

@andrhamm
Copy link

andrhamm commented May 7, 2018

There seems to be a dependency required for aws_signers_v4? Is that correct? It would be great if the documentation reflected the required dependencies and versions to use.

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.

4 participants