Skip to content

Update the Google Maps custom map style if changed after initialization#2017

Merged
rborn merged 1 commit intoreact-native-maps:masterfrom
mauritsd:dynamic-map-style
Apr 24, 2019
Merged

Update the Google Maps custom map style if changed after initialization#2017
rborn merged 1 commit intoreact-native-maps:masterfrom
mauritsd:dynamic-map-style

Conversation

@mauritsd
Copy link
Contributor

@mauritsd mauritsd commented Feb 7, 2018

Previously, the library would not pass on the updated style to native Google Maps if a change occurred after onMapReady had fired. The native side doesn't seem to mind this happening, although I did not test extensively.

@rborn
Copy link
Collaborator

rborn commented Feb 7, 2018

@mauritsd thanks for the PR
What do you mean by "not tested extensively" what were the situations you tested for? 🤗

@mauritsd
Copy link
Contributor Author

mauritsd commented Feb 7, 2018

By that I meant that I didn't test it on Android as the app that required this bit of functionality is iOS-only. On iOS everything works as expected. I'll see if I can add it to the example.

@rborn
Copy link
Collaborator

rborn commented Feb 7, 2018

Did you test both apple and google maps?
Also could you test on android too ?

🤓

@rborn
Copy link
Collaborator

rborn commented Mar 12, 2018

@mauritsd any update in this? 🤗

@rborn rborn closed this Apr 3, 2019
@rborn rborn reopened this Apr 3, 2019
@rborn
Copy link
Collaborator

rborn commented Apr 3, 2019

LGTM (except the outdated componenWillMount but it's outside the scope of this commit)

@alvelig @christopherdro 🐽

@rborn
Copy link
Collaborator

rborn commented Apr 23, 2019

@alvelig @christopherdro 🐽 how about this one?

@alvelig
Copy link
Contributor

alvelig commented Apr 24, 2019

LGTM

@rborn rborn merged commit 4c2d8e3 into react-native-maps:master Apr 24, 2019
pohnean pushed a commit to pohnean/react-native-maps that referenced this pull request Jul 28, 2020
pinpong pushed a commit to pinpong/react-native-maps that referenced this pull request Feb 28, 2025
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.

3 participants