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

Update installation.md #696

Merged
merged 2 commits into from
Oct 16, 2016

Conversation

securingsincity
Copy link
Contributor

No description provided.

@christopherdro
Copy link
Collaborator

Hey @securingsincity! Thanks for the PR.

I actually was looking at this the other day and am thinking it will be a good idea to add the documentation.

Note: As shown above, com.google.android.geo.API_KEY is the recommended metadata name for the API key. A key with this name can be used to authenticate to multiple Google Maps-based APIs on the Android platform, including the Google Maps Android API. For backwards compatibility, the API also supports the name com.google.android.maps.v2.API_KEY. This legacy name allows authentication to the Android Maps API v2 only. An application can specify only one of the API key metadata names. If both are specified, the API throws an exception.
Source: https://developers.google.com/maps/documentation/android-api/signup

Is there a chance you can update this PR to take the note above into account?

There are a few other notes we could add about setting up restricted usage where the apps package name and SHA-1 signing-certificate fingerprint need to be added to the list.

I can update the docs with this info after we get this merged in.

@christopherdro
Copy link
Collaborator

Thanks!

@christopherdro christopherdro merged commit 60afece into react-native-maps:master Oct 16, 2016
Exilz pushed a commit to archriss/react-native-maps that referenced this pull request Dec 9, 2016
* Update installation.md

* Adds note about usage of google api key.
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.

2 participants