-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Update api to match Cordova implementation #5
Conversation
return NativeCodePush.downloadUpdate(this) | ||
.then((downloadedPackage) => { | ||
return extend({}, downloadedPackage, local); | ||
}); |
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.
what happens if the download fails? cordova side has an error handller for this
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.
Since we return a promise, any errors thrown by downloadUpdate()
will be bubbled up to the calling code.
LGTM |
|
||
[CodePush loadBundle]; | ||
|
||
//resolve([NSNull null]); |
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.
remove?
@@ -1,4 +1,4 @@ | |||
/** | |||
/*y |
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.
?
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.
Accidental change. I'll revert.
minor comments, LGTM otherwise!! 👍 |
Update api to match Cordova implementation
…d_apps_v2 Support for brownfield apps v2
This is a pretty big refactor. Pulled out most of the package handling code to a distinct
CodePushPackage.m
file. Still a few pieces of polish I need to do on the Obj-C code, but I wanted to get this out for review since I've been working on it in private for so long.