Re: [iOS] Intent to delete AppDelegate remote and local notification code in 4.0

2015-02-11 Thread tommy-carlos williams
+1 The email currently says “If your app does not use the Apple Push Notification service, no action is required”, but it probably freaks new users out. Also, it might become a “thing” in the future. --  tommy-carlos williams On 12 February 2015 at 10:10:59, Shazron (shaz...@gmail.com)

[iOS] Intent to delete AppDelegate remote and local notification code in 4.0

2015-02-11 Thread Shazron
See: https://github.com/apache/cordova-ios/blob/338ee71f966ab7fdc1ccde02e5086febbc82b70c/bin/templates/project/__PROJECT_NAME__/Classes/AppDelegate.m#L110-L135 I'd rather do this in 3.8.0, but we would break things for users, so in 4.0 it goes. Related issue:

Re: [iOS] Intent to delete AppDelegate remote and local notification code in 4.0

2015-02-11 Thread Shazron
Great idea, let's do that. On Wednesday, February 11, 2015, Andrew Grieve agri...@chromium.org wrote: Sounds good. Since we're making a 4.0 change here anyways, might be a good idea to empty out that template file and instead have it extend a base class from CordovaLib (this is what Android

Re: [iOS] Intent to delete AppDelegate remote and local notification code in 4.0

2015-02-11 Thread Andrew Grieve
Sounds good. Since we're making a 4.0 change here anyways, might be a good idea to empty out that template file and instead have it extend a base class from CordovaLib (this is what Android does). Main reason is that the file will be updated when swapping in new versions of CordovaLib. On Wed,