Make location block to return nullable dictionary

[GRiMe2D]

Summary

Amplitude v7.0.0 added location block for providing user location manually. But location isn't available instantaneously. For some moment location will be unknown. But new added location block doesn't mark it nullable however it checks for nil while accepting it. Can't return nil in swift

Amplitude-iOS/Sources/Amplitude/Amplitude.h

Line 32 in d7f81d3

typedef NSDictionary *_Nonnull (^AMPLocationInfoBlock)(void);

Amplitude-iOS/Sources/Amplitude/Amplitude.m

Lines 664 to 669 in d7f81d3

	if ([_appliedTrackingOptions shouldTrackLatLng] && self.locationInfoBlock != nil) {
	NSDictionary *location = self.locationInfoBlock();
	if (location != nil) {
	[apiProperties setValue:location forKey:@"location"];
	}
	}

Proposed

 NS_ASSUME_NONNULL_BEGIN
 
 typedef NSString *_Nonnull (^AMPAdSupportBlock)(void);
-typedef NSDictionary *_Nonnull (^AMPLocationInfoBlock)(void);
+typedef NSDictionary *_Nullable (^AMPLocationInfoBlock)(void);

[haoliu-amp]

Thanks for reporting the issue!
We released v7.0.1 to address it!
https://github.com/amplitude/Amplitude-iOS/releases/tag/v7.0.1

[john-mejia]

@haoliu-amp would it be possible to call out breaking changes in future release notes? This caused a breaking change in our application and we had to dig through the PR/code to get an understanding of exactly what went wrong.

[haoliu-amp]

@john-mejia Sorry for your trouble with new changes. We did mark it to be BREAKING CHANGES in v7.0.0. (We bumped it to major version 7 since it's not back compatible change.) https://github.com/amplitude/Amplitude-iOS/releases/tag/v7.0.0

[john-mejia]

woah,. my apologies - we jumped straight to .1 and missed the original update!