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
[TIMOB-23888] iOS: Use asynchronous openURL API on iOS 10+ #9759
Conversation
iphone/Classes/PlatformModule.m
Outdated
NSMutableDictionary *options = [NSMutableDictionary dictionary]; | ||
|
||
if ([args count] > 0) { | ||
if ([[args objectAtIndex:1] isKindOfClass:[NSDictionary class]]) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In case of count = 1, [args objectAtIndex:1] will fail. Due to this first test case is failing.
* UIApplicationOpenURLOptionsSourceApplicationKey | ||
* UIApplicationOpenURLOptionsAnnotationKey | ||
* UIApplicationOpenURLOptionsOpenInPlaceKey | ||
* UIApplicationOpenURLOptionUniversalLinksOnly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should give the detail about the values of these string-keys. e.g For UIApplicationOpenURLOptionUniversalLinksOnly value should be Bool or give an apple link here.
And normally we map apple keys with Titanium keys. Isn't it good to use titanium keys?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reason I've not made this a constant is because we cannot use constants as dictionary-keys, so the dev would need to declare a variable first and then use that one as a key. I'll add more infos about the constants though!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR passed.
Generated by 🚫 dangerJS |
JIRA: https://jira.appcelerator.org/browse/TIMOB-23888