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

mock location #476

Merged
merged 4 commits into from
Jun 15, 2017
Merged

mock location #476

merged 4 commits into from
Jun 15, 2017

Conversation

cammace
Copy link
Contributor

@cammace cammace commented Jun 8, 2017

This moves the mockLocationEngine to a more central location rather than including it in several different projects and having to maintain them independently.

@cammace cammace added this to the v2.2.0 milestone Jun 8, 2017
@cammace cammace self-assigned this Jun 8, 2017
@cammace cammace requested a review from zugaldia June 8, 2017 15:32
@cammace cammace changed the title Cam mock location mock location Jun 8, 2017
@zugaldia
Copy link
Member

zugaldia commented Jun 8, 2017

Sounds good. Let's remove it then from any other places it's being used like mapbox/app/src/main/java/com/mapbox/services/android/testapp/location/MockLocationEngine.java.

@cammace
Copy link
Contributor Author

cammace commented Jun 8, 2017

Removed the old mock location and changed locationEngineActivity to use the new one.

@cammace cammace merged commit fde68c0 into master Jun 15, 2017
@cammace cammace deleted the cam-mock-location branch June 15, 2017 19:11
@zugaldia zugaldia mentioned this pull request Jun 20, 2017
10 tasks
@cammace cammace mentioned this pull request Jul 20, 2017
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants