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

Timob 18854 : Add Delegate & Reply for openParentApplication to Titanium #6803

Closed
wants to merge 4 commits into from
Closed

Timob 18854 : Add Delegate & Reply for openParentApplication to Titanium #6803

wants to merge 4 commits into from

Conversation

benbahrenburg
Copy link
Contributor

This PR relates to ticket TIMOB-18854

Included in this PR is:

1.Addition of the required Application Delegate
2.Ti.App.iOS notification with information provided by the calling WatchKit Extension
3.sendWatchExtensionReply method to reply back to the openParentApplication

A detail of all the changes included in this PR is available here

Add support for openParentApplication which requires handleWatchKitExtensionRequest
ENSURE_ARG_COUNT(args, kArgCount);

NSString *key = [TiUtils stringValue:[args objectAtIndex:kArgKey]];
NSDictionary *userInfo = [args objectAtIndex:kArgUserInfo];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The kArgUserInfo value is 1 so it should be inside the if([args count] > 1) block

@pec1985
Copy link
Contributor

pec1985 commented Apr 30, 2015

@benbahrenburg thanks for this PR. I left a minor comment that needs to be addressed in order to avoid crashes.
One thing that's missing is documentation. Please look into the apidoc/Titanium/App/iOS/iOS.yml
Thanks!

@benbahrenburg
Copy link
Contributor Author

@pec1985 documentation added. Dictionary argument inlined.

@pec1985
Copy link
Contributor

pec1985 commented May 7, 2015

Closing PR, fixed a minor docuemtnation issue, new PR with @benbahrenburg's changes here: #6826

Thanks Ben!

@pec1985 pec1985 closed this May 7, 2015
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

2 participants