Skip to content

Support Android LiteMode#546

Merged
jrichardlai merged 1 commit intoreact-native-maps:masterfrom
tt-sport-mobile:master
Sep 20, 2016
Merged

Support Android LiteMode#546
jrichardlai merged 1 commit intoreact-native-maps:masterfrom
tt-sport-mobile:master

Conversation

@rops
Copy link
Copy Markdown
Contributor

@rops rops commented Sep 7, 2016

Following discussion #300

@spikebrehm
Copy link
Copy Markdown

Thanks, this looks good!

Can you squash the commits first? There's 18 of them, with a lot of merge commits.

cc: @felipecsl to look at Android code.


private final ReactApplicationContext appContext;

protected GoogleMapOptions googleMapOptions;
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.

final

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.

can't be final as AirMapLiteManager overrides it

@felipecsl
Copy link
Copy Markdown
Contributor

android code lgtm after the 2 comments are addressed

@jrichardlai
Copy link
Copy Markdown
Contributor

Should we add an example for liteMode?

@spikebrehm
Copy link
Copy Markdown

Should we add an example for liteMode?

Great idea! Let's add that before merging.

@rops
Copy link
Copy Markdown
Contributor Author

rops commented Sep 17, 2016

Adding Examples now

@rops
Copy link
Copy Markdown
Contributor Author

rops commented Sep 17, 2016

Added docs and examples

@rops
Copy link
Copy Markdown
Contributor Author

rops commented Sep 20, 2016

Can someone take another quick look? I guess it's kind of ready to get in?

@jrichardlai
Copy link
Copy Markdown
Contributor

tried the example, looks good :) 👍

@jrichardlai jrichardlai merged commit 70a6c8a into react-native-maps:master Sep 20, 2016
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.

4 participants