General question to everybody: The patch requires the iOS 8 SDK (i.e. Xcode 6). We require Xcode 6 in cordova-ios 3.7.0 and there was consensus on it. However, the plugin.xml itself does not have an engine tag to specify what it supports.
Question: Do we have engine tag support in the CLI, where it won't install a plugin if the requirements are not met? If not, we would have to do a #ifdef __IPHONE_8_0 macro in the code. On Fri, Nov 14, 2014 at 11:18 AM, Shazron <shaz...@gmail.com> wrote: > Thanks Julio! > I'll comment on the PR itself. > > Shaz > > On Fri, Nov 14, 2014 at 8:42 AM, Ian Clelland <iclell...@chromium.org> > wrote: > >> Well, I think what you've just done (pinging the list) is pretty close to >> the right next step. >> >> You should probably assign the issue back to Shaz with a note directed to >> him asking him to take a look and merge it in. (I'd merge it, but I haven't >> been following iOS 8 development closely enough to judge its correctness) >> >> And thanks for taking it on! >> >> Ian >> >> >> On Fri Nov 14 2014 at 2:32:14 AM julio cesar sanchez < >> jcesarmob...@gmail.com> wrote: >> >>> CB-7734 (https://issues.apache.org/jira/browse/CB-7734) was reported >>> and I >>> asked if it could be assigned to me. >>> Shazron assigned to me and I fixed it with this pull request >>> https://github.com/apache/cordova-plugin-dialogs/pull/39 >>> >>> It's been 3 weeks and it hasn't been merged, so I don't know if I have to >>> do something else, it's the first issue assigned to me. >>> >> >