Skip to content

[iOS] Added Google Maps Circle, Polygon, Polyline, MapType Support#722

Merged
spikebrehm merged 4 commits intoreact-native-maps:masterfrom
nitaliano:google-maps-ios-shapes
Oct 27, 2016
Merged

[iOS] Added Google Maps Circle, Polygon, Polyline, MapType Support#722
spikebrehm merged 4 commits intoreact-native-maps:masterfrom
nitaliano:google-maps-ios-shapes

Conversation

@nitaliano
Copy link
Copy Markdown
Contributor

Needed the ability to draw on maps and switch the base map using the Google Maps SDK so I took a stab at adding this support.

@volkangurel
Copy link
Copy Markdown

👍

Copy link
Copy Markdown
Contributor

@gilbox gilbox left a comment

Choose a reason for hiding this comment

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

this is awesome, thanks for taking this on! just one question.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

did you consider keeping this check in case someone is still using mapkit?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@gilbox good point! I can add this check back and base it off the platform and provider

@nitaliano nitaliano force-pushed the google-maps-ios-shapes branch from 9575256 to 0ce4b3e Compare October 24, 2016 21:51
@nitaliano
Copy link
Copy Markdown
Contributor Author

Added the check back

@gilbox
Copy link
Copy Markdown
Contributor

gilbox commented Oct 24, 2016

this lgtm.. @spikebrehm wanna take a peek?

@nitaliano nitaliano changed the title [iOS] Added Google Maps Polygon, Polyline, MapType Support [iOS] Added Google Maps Circle, Polygon, Polyline, MapType Support Oct 25, 2016
@nitaliano
Copy link
Copy Markdown
Contributor Author

@gilbox @spikebrehm Just added circle support

@enagy27
Copy link
Copy Markdown

enagy27 commented Oct 25, 2016

👍

@Stonelinks
Copy link
Copy Markdown

Stonelinks commented Oct 26, 2016

👍
(In case you were wondering, Airware has started playing around with this)

@manelpb
Copy link
Copy Markdown

manelpb commented Oct 26, 2016

This is perfect! Guys please merge this pull-request!!

@spikebrehm
Copy link
Copy Markdown

LGTM! Thanks 🍻

@spikebrehm spikebrehm merged commit 2486841 into react-native-maps:master Oct 27, 2016
pinpong pushed a commit to pinpong/react-native-maps that referenced this pull request Feb 28, 2025
…-ios-shapes

[iOS] Added Google Maps Circle, Polygon, Polyline, MapType Support
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.

7 participants