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

New Feature: Support for Google Map Config (mapId, liteMode) #4922

Merged

Conversation

salah-ghanim
Copy link
Collaborator

Does any other open PR do the same thing?

No

What issue is this PR fixing?

Google Maps introduced a new styling capability via google cloud, this library didn't support any kind of initialisation on view creation before, and depended even on creating a custom module just to pass an initial property for Android: liteMode

This is now fixed and initial properties can be passed

supported properties: googleMapId (iOS, Android) , liteMode (Android only)

How did you test this PR?

Physically on android and iOS devices, the values are being setup correctly.

Unfortunately for iOS without updating to google-maps SDK v 8.2 the style won't be visible .. waiting on google

salah ghanim added 2 commits December 23, 2023 15:47
- Android: LiteMode and mapID as InitProps
- Android: remove unnecessary module for MapLite
- Examples: LiteMode working with initialProps
formatting: whitespace cleanup
- iOS: code works to pass mapID to googleMaps
- iOS: swizzling RCTComponentData to add support for initialProps
- iOS: Any RCTViewManager will automatically have initialProps autofilled if property is available
- remove unused import from MapView
@GETAWAY-Dev GETAWAY-Dev self-requested a review January 1, 2024 11:32
Copy link
Collaborator

@GETAWAY-Dev GETAWAY-Dev left a comment

Choose a reason for hiding this comment

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

LGTM

@salah-ghanim salah-ghanim merged commit 77610e9 into react-native-maps:master Jan 2, 2024
4 checks passed
@react-native-maps-bot
Copy link
Collaborator

🎉 This PR is included in version 1.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@charneykaye
Copy link

charneykaye commented May 7, 2024

🤦 still waiting on googlemaps/google-maps-ios-utils#451 to be merged for iOS support

@demoox
Copy link

demoox commented Jun 14, 2024

Hi, It's operational or not yet ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants