Skip to content

Fix/nullability#614

Merged
spikebrehm merged 2 commits intoreact-native-maps:masterfrom
3sidedcube:fix/nullability
Sep 27, 2016
Merged

Fix/nullability#614
spikebrehm merged 2 commits intoreact-native-maps:masterfrom
3sidedcube:fix/nullability

Conversation

@simonmitchell
Copy link
Copy Markdown
Contributor

Fixes compiler error with trying to specify nullability on a BOOL

Tim Matthews and others added 2 commits September 26, 2016 10:54
* commit '7a5083cbb820faec0bb684c010e15c222ce7e533': (22 commits)
  Add an options argument
  Update README with example
  Add edgePadding example
  Add default arguments to fitToCoordinates
  -Set constant for baseMapPadding used in newLatLngBounds calls -Remove extraenous comment
  -Port fitToCoordinates functionality over to Android
  Add example
  Update docs
  Add fitToCoordinates method
  Fix list of examples on Android
  Support Google Maps on iOS (react-native-maps#548)
  fix react-native-maps#453
  Make MAP_TYPES constant, don't pass `none` mapType to iOS
  Fix some changes missed in rebase
  renamed the 'url' property to 'urlTemplate' for consistency with ios
  added support for tile overlays (MKTileOverlay)
  changed url template to use {x} {y} {z} pattern replacement
  initial support for tile overlays
  Updated Mapview onRegionChange events notes
  Adding support for Android lite mode
  ...
@spikebrehm
Copy link
Copy Markdown

@simonmitchell in #611 you said that this fix is necessary to get the project to build with CocoaPods. How does this change affect CocoaPods builds?

In this commit, @naoufal added default values to the fitToCoordinates options, so animated should not be null unless you're manually passing it in, right?

@gilbox
Copy link
Copy Markdown
Contributor

gilbox commented Sep 27, 2016

@spikebrehm i had to make this same fix for a from-scratch simple non-cocoapods test project.

@spikebrehm
Copy link
Copy Markdown

👍

@spikebrehm spikebrehm merged commit 586414e into react-native-maps:master Sep 27, 2016
@simonmitchell
Copy link
Copy Markdown
Contributor Author

simonmitchell commented Sep 27, 2016

Yeah sorry for the confusion, wasn't related to the CocoaPods issue, was just telling that person that even after I used CocoaPods I still needed to make changes. But yeah, agree that animated could be null, only problem being c primitives can't be null in Objective-C ;)

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