Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[android] LatLng bounds cannot be wrapped #13419

Merged
merged 2 commits into from
Dec 12, 2018
Merged

[android] LatLng bounds cannot be wrapped #13419

merged 2 commits into from
Dec 12, 2018
+430 −482

Conversation

osana
Copy link
Contributor

@osana osana commented Nov 20, 2018

closes #13149

@osana osana added Android Mapbox Maps SDK for Android ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold SEMVER-MAJOR Requires a major release according to Semantic Versioning rules labels Nov 20, 2018
@osana osana force-pushed the osana-lat-lng-donotwrap branch 4 times, most recently from d69f9a5 to 4f6b495 Compare November 21, 2018 00:07
@LukasPaczos
Copy link
Contributor

When this is ready, would you mind verifying that it fixes #13186 as well?

@osana osana force-pushed the osana-lat-lng-donotwrap branch 3 times, most recently from e6935e9 to 356acee Compare November 27, 2018 02:33
@tobrun tobrun added the beta blocker Blocks the next beta release label Dec 7, 2018
@osana osana force-pushed the osana-lat-lng-donotwrap branch 2 times, most recently from 58f7600 to 6204f9e Compare December 11, 2018 15:19
@LukasPaczos LukasPaczos mentioned this pull request Dec 11, 2018
Merged
@osana osana force-pushed the osana-lat-lng-donotwrap branch 8 times, most recently from 00602db to fdbeece Compare December 11, 2018 19:20
@osana osana force-pushed the osana-lat-lng-donotwrap branch 3 times, most recently from ac2c5c0 to 324127d Compare December 11, 2018 21:57
Don't wrap LatLngBounds (#13540)
4c45123 
* [android] - don't wrap LatLngBounds

* Update GeometryConstants.java

* Update NativeMapTest
@osana osana removed the ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold label Dec 12, 2018
@osana osana mentioned this pull request Dec 12, 2018
Closed
LukasPaczos
LukasPaczos approved these changes Dec 12, 2018
View reviewed changes
Copy link
Contributor

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@LukasPaczos LukasPaczos merged commit 6aea5b5 into master Dec 12, 2018
@LukasPaczos LukasPaczos deleted the osana-lat-lng-donotwrap branch December 12, 2018 14:06
LukasPaczos pushed a commit that referenced this pull request Dec 12, 2018
@osana @LukasPaczos
[android] LatLng bounds cannot be wrapped (#13419)
1f07df2 
(cherry picked from commit 6aea5b5)
@LukasPaczos LukasPaczos mentioned this pull request Dec 12, 2018
Merged
LukasPaczos pushed a commit that referenced this pull request Dec 12, 2018
@osana
[android] LatLng bounds cannot be wrapped (#13419)
dbb7f44 
(cherry picked from commit 6aea5b5)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@LukasPaczos LukasPaczos LukasPaczos approved these changes

@tobrun tobrun Awaiting requested review from tobrun

Assignees
No one assigned
Labels
Android Mapbox Maps SDK for Android beta blocker Blocks the next beta release SEMVER-MAJOR Requires a major release according to Semantic Versioning rules
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove LatLng wrapping
3 participants
@osana @LukasPaczos @tobrun