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

Add documentation on API KEY usage. #49

Merged
merged 2 commits into from
May 16, 2016
Merged

Add documentation on API KEY usage. #49

merged 2 commits into from
May 16, 2016

Conversation

celsom3
Copy link
Contributor

@celsom3 celsom3 commented May 16, 2016

Included an edit to the example in the README. Had to find this info in
a closed issue and thought it might be useful for others to have it
available in the README.

Included an edit to the example in the README. Had to find this info in
a closed issue and thought it might be useful for others to have it
available in the README.
@MicheleBertoli
Copy link
Owner

MicheleBertoli commented May 16, 2016

Awesome, thanks @celsom3!

We've got a couple of issues (#45 #43) where people were asking how to set the api key recently.
I was waiting for some feedbacks on #29 to write some proper docs but your PR is great for now.

If you could just fix the indentation please, that would be perfect:
params={{v: '3.exp', key: 'YOUR_API_KEY'}}

@celsom3
Copy link
Contributor Author

celsom3 commented May 16, 2016

Glad it's useful. I went ahead and changed the indentation.

@MicheleBertoli MicheleBertoli merged commit fcb480f into MicheleBertoli:master May 16, 2016
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.

None yet

2 participants