-
Notifications
You must be signed in to change notification settings - Fork 394
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
Upgrade #65
base: master
Are you sure you want to change the base?
Upgrade #65
Conversation
Minor changes to UI. Added API key to manifest. Updated prediction text to show both lines, makes it more clear.
Added toolbar.
Thanks. Could you check to see what's actualy casuing the build to fail. |
<meta-data | ||
android:name="com.google.android.geo.API_KEY" | ||
android:value="API_KEY_HERE"/> | ||
android:value="AIzaSyDv8K5B4U6M78r8JFufD0fTFpyycW7cmMk"/> |
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 correct? It should not be like before?
No, everyone puts their own API key there. Yours is private so it shouldn't be shared. |
TY |
The build failed. check to see why it failed here and how it can be resolved https://travis-ci.org/jd-alexander/Google-Directions-Android/builds/167146779 |
…s only the initial segment. waypoints were not supported. I'll have to see if this affects me for transit. seems not so far.
Hi Alexander,
I took the liberty of upgrading the library sample to the latest version of Google tools (gradle, SDKs and libraries, and so on).
Everything seems to still work.
Could you have a look and merge if everything is ok? I want to use your library in the future.
Regards,
Radu