Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Isochrone API refactor to use Java SDK wrapper #1029

Merged
merged 1 commit into from
May 3, 2019

Conversation

langsmith
Copy link
Contributor

This pr refactors IsochroneActivity to use the Java SDK's Isochrone API support that landed in the 4.6.0 Java SDK release.

@langsmith
Copy link
Contributor Author

@osana , this pr is blocked by a bug in the Isochrone API code (my fault). I'm about to post an issue ticket in the Java SDK.

@langsmith
Copy link
Contributor Author

This pr is waiting on mapbox/mapbox-java#997 landing, a new version of the Java SDK being released, and then bumping this app's Java SDK dependency to the new number....

@langsmith
Copy link
Contributor Author

langsmith commented Apr 23, 2019

Ok, the refactoring on this pr gets IsochroneActivity working as intended.

56605678-84a0da00-65b9-11e9-8dd6-cf0cc7a19db6

56605679-84a0da00-65b9-11e9-8a3e-dbb4227472ed

@langsmith langsmith force-pushed the ls-isochrone-refactor-to-use-Java-wrapper branch 5 times, most recently from cfa1a6b to 1c83370 Compare April 27, 2019 00:49
@langsmith langsmith force-pushed the ls-isochrone-refactor-to-use-Java-wrapper branch from 1c83370 to 438861d Compare April 30, 2019 16:21
@langsmith
Copy link
Contributor Author

Ok, this is ready for review

@langsmith langsmith merged commit 162f46d into master May 3, 2019
@langsmith langsmith deleted the ls-isochrone-refactor-to-use-Java-wrapper branch May 3, 2019 14:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants