Skip to content

Protect against null Location in FusedLocationSource#3345

Merged
christopherdro merged 2 commits intoreact-native-maps:masterfrom
netshade:protect-against-null-location-in-fused-location-source
Mar 17, 2020
Merged

Protect against null Location in FusedLocationSource#3345
christopherdro merged 2 commits intoreact-native-maps:masterfrom
netshade:protect-against-null-location-in-fused-location-source

Conversation

@netshade
Copy link
Copy Markdown
Contributor

Does any other open PR do the same thing?

No

What issue is this PR fixing?

#3344

How did you test this PR?

Tested on Android Emulator w/ Detox on a CI server where this was previously occurring. The crash does not recur.

@rborn
Copy link
Copy Markdown
Collaborator

rborn commented Mar 17, 2020

@netshade 😻

LGTM @christopherdro 🐽

@christopherdro christopherdro merged commit eeb13bf into react-native-maps:master Mar 17, 2020
@netshade netshade deleted the protect-against-null-location-in-fused-location-source branch March 18, 2020 14:41
pinpong pushed a commit to pinpong/react-native-maps that referenced this pull request Feb 28, 2025
…ps#3345)

* protect against null location in map view

* Update FusedLocationSource.java

Co-authored-by: Christopher Dro <hello@blick-labs.com>
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