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

0.4 stable #276

Merged
merged 3 commits into from
Jun 10, 2016
Merged

0.4 stable #276

merged 3 commits into from
Jun 10, 2016

Conversation

locly
Copy link

@locly locly commented Jun 8, 2016

Fix for#165 and #271 - Android 6.0 crashing on devices such as Xperia Z5 Compact, Z5 Bond and others.

It passes the activity to the package, manager and view so that it can be used in place of the app context when initialising the MapView component.

The new code as also been tested on the following devices to ensure it continues to work as before:
Nexus 4 (Android 5), Nexus 5 (Android 6), Nexus 6P (Android 6)

It requires a change to an app's MainActivity.java to pass this when creating the AirPackage: new AirPackage(this) rather than new AirPackage() in the current version.

@jrichardlai
Copy link
Contributor

Thanks for finding the issue and fixing it! I think the example app will need to be updated too.

@christopherdro
Copy link
Collaborator

@locly Nice! Yea, could you update the example to pass this when declaring the package. Thanks!

@locly
Copy link
Author

locly commented Jun 9, 2016

Yes - have updated the example to declare the package with new AirPackage(this)

@jrichardlai
Copy link
Contributor

@locly looks good! thanks for the PR merging

@jrichardlai jrichardlai merged commit 5a2ad20 into react-native-maps:0.4-stable Jun 10, 2016
@jrichardlai
Copy link
Contributor

@locly just realized this was for the 0.4-stable branch. I think this should also work for master, any reason not too?
I can do it for master unless you want to tackle this.

jrichardlai added a commit to jrichardlai/react-native-maps that referenced this pull request Jun 13, 2016
christopherdro pushed a commit that referenced this pull request Jun 14, 2016
@locly locly mentioned this pull request Jul 7, 2016
BigPun86 pushed a commit to BigPun86/react-native-maps that referenced this pull request Dec 1, 2016
markdgo pushed a commit to markdgo/react-native-maps that referenced this pull request Mar 15, 2021
MarcoAntonioAG pushed a commit to MarcoAntonioAG/React-map that referenced this pull request Mar 31, 2022
joshpeterson30489 added a commit to joshpeterson30489/maps-develop-with-react-native that referenced this pull request Sep 30, 2022
superstar1205 added a commit to superstar1205/Map-ReactNative that referenced this pull request Dec 26, 2022
anthony0506 added a commit to anthony0506/react-native-maps that referenced this pull request Mar 19, 2023
johney6767 pushed a commit to johney6767/Map-ReactNative that referenced this pull request May 31, 2023
GoldenDragon0710 added a commit to GoldenDragon0710/google-map-react-native that referenced this pull request Nov 27, 2023
PainStaker0331 pushed a commit to PainStaker0331/react-native-maps that referenced this pull request Mar 3, 2024
Super-CodeKing added a commit to Super-CodeKing/react_native_map that referenced this pull request Apr 26, 2024
fairskyDev0201 pushed a commit to fairskyDev0201/react-native-maps that referenced this pull request Apr 29, 2024
DavidLee0501 added a commit to DavidLee0501/React-Native-Map that referenced this pull request Aug 12, 2024
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.

None yet

3 participants