[GitHub] cordova-plugin-wkwebview-engine pull request #22: CB-11937 - Travis CI build...

2017-01-25 Thread shazron
Github user shazron closed the pull request at: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/22 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] cordova-plugin-wkwebview-engine issue #22: CB-11937 - Travis CI build failin...

2017-01-25 Thread shazron
Github user shazron commented on the issue: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/22 Fixed in #28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enable

[GitHub] cordova-plugin-wkwebview-engine issue #25: CB-12317: Allow to configure navi...

2017-01-25 Thread shazron
Github user shazron commented on the issue: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/25 I've fixed the CI problem and would like this PR to be green before I pull it in. Can you rebase? It should re-test the PR automatically after. --- If your project is set up

[GitHub] cordova-plugin-wkwebview-engine issue #27: CB-12297 Support WKProcessPool fo...

2017-01-25 Thread shazron
Github user shazron commented on the issue: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/27 I've fixed the CI problem and would like this PR to be green before I pull it in. Can you rebase? It should re-test the PR automatically after. --- If your project is set up

[GitHub] cordova-plugin-file-transfer issue #174: CB-12399: (Android) Fix bug with We...

2017-01-25 Thread cordova-qa
Github user cordova-qa commented on the issue: https://github.com/apache/cordova-plugin-file-transfer/pull/174 Cordova CI Build has completed successfully. **Commit** - [Link](https://github.com/apache/cordova-plugin-file-transfer/pull/174/commits/ab268e0605c55c47b3931029b

[GitHub] cordova-plugin-file-transfer pull request #174: CB-12399: (Android) Fix bug ...

2017-01-25 Thread Lemon-King
GitHub user Lemon-King opened a pull request: https://github.com/apache/cordova-plugin-file-transfer/pull/174 CB-12399: (Android) Fix bug with WebView & CookieManager cookies ### Platforms affected Android ### What does this PR do? Resolves the issue where only WebVi

[GitHub] cordova-plugin-wkwebview-engine pull request #28: Cb 11937 - Travis CI Build...

2017-01-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, o

[GitHub] cordova-plugin-screen-orientation issue #10: Update screen orientation plugi...

2017-01-25 Thread cordova-qa
Github user cordova-qa commented on the issue: https://github.com/apache/cordova-plugin-screen-orientation/pull/10 Cordova CI Build has one or more failures. **Commit** - [Link](https://github.com/apache/cordova-plugin-screen-orientation/pull/10/commits/71075a1f32651befd0

[GitHub] cordova-plugin-screen-orientation issue #10: Update screen orientation plugi...

2017-01-25 Thread cordova-qa
Github user cordova-qa commented on the issue: https://github.com/apache/cordova-plugin-screen-orientation/pull/10 Cordova CI Build has one or more failures. **Commit** - [Link](https://github.com/apache/cordova-plugin-screen-orientation/pull/10/commits/083578eaa87c696f55

Re: CB-11602: (android) Adds onRestart event support

2017-01-25 Thread julio cesar sanchez
+1, iOS needs the launch images even if you don't install the plugin and the launch storyboard has some of the logic in cordova-ios 2017-01-25 22:31 GMT+01:00 Filip Maj : > +1 ! > > On Wed, Jan 25, 2017 at 11:35 AM, Kerri Shotts > wrote: > > +1 to moving splash screen into platforms. > > > > > >

[GitHub] cordova-lib issue #499: Cb 11960

2017-01-25 Thread stevengill
Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/499 ya On Wed, Jan 25, 2017 at 2:35 PM, Audrey wrote: > *@audreyso* commented on this pull request. > -- > > In cordova-lib/spec-cordova/f

[GitHub] cordova-lib pull request #499: Cb 11960

2017-01-25 Thread audreyso
Github user audreyso commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/499#discussion_r97895425 --- Diff: cordova-lib/spec-cordova/fixtures/basePkgJson4/package.json --- @@ -5,7 +5,7 @@ "main": "index.js", "dependencies": {

[GitHub] cordova-lib pull request #499: Cb 11960

2017-01-25 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/499#discussion_r97894467 --- Diff: cordova-lib/spec-cordova/pkgJson-restore.spec.js --- @@ -680,10 +681,10 @@ describe('update config.xml to include platforms in pkg.json', fu

[GitHub] cordova-lib pull request #499: Cb 11960

2017-01-25 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/499#discussion_r97894300 --- Diff: cordova-lib/spec-cordova/fixtures/basePkgJson4/package.json --- @@ -5,7 +5,7 @@ "main": "index.js", "dependencies": {

[GitHub] cordova-lib pull request #499: Cb 11960

2017-01-25 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/499#discussion_r97894545 --- Diff: cordova-lib/spec-cordova/pkgJson-restore.spec.js --- @@ -697,18 +698,18 @@ describe('update config.xml to include platforms in pkg.json', fu

[GitHub] cordova-lib pull request #499: Cb 11960

2017-01-25 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/499#discussion_r97894565 --- Diff: cordova-lib/spec-cordova/pkgJson-restore.spec.js --- @@ -697,18 +698,18 @@ describe('update config.xml to include platforms in pkg.json', fu

[GitHub] cordova-lib pull request #499: Cb 11960

2017-01-25 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/499#discussion_r97894506 --- Diff: cordova-lib/spec-cordova/pkgJson-restore.spec.js --- @@ -697,18 +698,18 @@ describe('update config.xml to include platforms in pkg.json', fu

Re: CB-11602: (android) Adds onRestart event support

2017-01-25 Thread Filip Maj
+1 ! On Wed, Jan 25, 2017 at 11:35 AM, Kerri Shotts wrote: > +1 to moving splash screen into platforms. > > > ~ Kerri > >> On Jan 25, 2017, at 01:09, Simon MacDonald wrote: >> >> I would be happy to see Splash Screen go back to the platforms as well. >> >> Simon Mac Donald >> http://simonmacdona

[GitHub] cordova-plugin-media issue #122: CB-12157: (android): Fix java.lang.NullPoin...

2017-01-25 Thread ryanagler
Github user ryanagler commented on the issue: https://github.com/apache/cordova-plugin-media/pull/122 My app also crashes from this issue. Applying the changes from frankkoenigstein fixes it. Please merge to main! --- If your project is set up for it, you can reply to this email an

[GitHub] cordova-plugin-screen-orientation issue #10: Update screen orientation plugi...

2017-01-25 Thread cordova-qa
Github user cordova-qa commented on the issue: https://github.com/apache/cordova-plugin-screen-orientation/pull/10 Cordova CI Build has one or more failures. **Commit** - [Link](https://github.com/apache/cordova-plugin-screen-orientation/pull/10/commits/68693cf017440e5931

Re: CB-11602: (android) Adds onRestart event support

2017-01-25 Thread Kerri Shotts
+1 to moving splash screen into platforms. ~ Kerri > On Jan 25, 2017, at 01:09, Simon MacDonald wrote: > > I would be happy to see Splash Screen go back to the platforms as well. > > Simon Mac Donald > http://simonmacdonald.com > > On Tue, Jan 24, 2017 at 10:56 PM, Joe Bowser wrote: > >> O

[GitHub] cordova-plugin-screen-orientation pull request #10: Update screen orientatio...

2017-01-25 Thread maverickmishra
GitHub user maverickmishra opened a pull request: https://github.com/apache/cordova-plugin-screen-orientation/pull/10 Update screen orientation plugin to w3 spec ### Platforms affected iOS, Android,Windows ### What does this PR do? This PR makes the plugin

[GitHub] cordova-cli issue #266: CB-12008 : made autosave the default for platform an...

2017-01-25 Thread stevengill
Github user stevengill commented on the issue: https://github.com/apache/cordova-cli/pull/266 Here is the discussion for the proposal: https://github.com/cordova/cordova-discuss/pull/61 --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cordova-cli issue #266: CB-12008 : made autosave the default for platform an...

2017-01-25 Thread brodybits
Github user brodybits commented on the issue: https://github.com/apache/cordova-cli/pull/266 I hope to see this integrated soon. I have seen way too many new users keeping old plugins in git instead of just adding them to config.xml. --- If your project is set up for it, you can repl

[GitHub] cordova-cli issue #255: CLI Refactor

2017-01-25 Thread brodybits
Github user brodybits commented on the issue: https://github.com/apache/cordova-cli/pull/255 Status? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] cordova-windows issue #213: CB-12163 Make resource-file copy files again

2017-01-25 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the issue: https://github.com/apache/cordova-windows/pull/213 @ktop, done! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and w

[GitHub] cordova-windows pull request #213: CB-12163 Make resource-file copy files ag...

2017-01-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-windows/pull/213 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] cordova-windows pull request #222: Upgrade cordova-common to 2.0.0

2017-01-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-windows/pull/222 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] cordova-windows issue #222: Upgrade cordova-common to 2.0.0

2017-01-25 Thread codecov-io
Github user codecov-io commented on the issue: https://github.com/apache/cordova-windows/pull/222 ## [Current coverage](https://codecov.io/gh/apache/cordova-windows/pull/222?src=pr) is 76.93% (diff: 100%) > Merging [#222](https://codecov.io/gh/apache/cordova-windows/pull/222?src=

[GitHub] cordova-windows pull request #222: Upgrade cordova-common to 2.0.0

2017-01-25 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-windows/pull/222 Upgrade cordova-common to 2.0.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/vladimir-kotikov/cordova-windows master Alter

[GitHub] cordova-windows issue #213: CB-12163 Make resource-file copy files again

2017-01-25 Thread ktop
Github user ktop commented on the issue: https://github.com/apache/cordova-windows/pull/213 @daserge Now that there has been a cordova-common release, can we refresh cordova-common in cordova-windows and get this merged? --- If your project is set up for it, you can reply to thi

[GitHub] cordova-docs pull request #678: Added nougat support version

2017-01-25 Thread victorsosa
GitHub user victorsosa opened a pull request: https://github.com/apache/cordova-docs/pull/678 Added nougat support version 6.0.0 Added nougat support version ### Platforms affected ### What does this PR do? ### What testing has been done

[GitHub] cordova-lib issue #515: CB-12383 Fix cordova_plugins metadata definition

2017-01-25 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the issue: https://github.com/apache/cordova-lib/pull/515 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] cordova-lib pull request #515: CB-12383 Fix cordova_plugins metadata definit...

2017-01-25 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/515#discussion_r97743108 --- Diff: cordova-lib/src/plugman/browserify.js --- @@ -157,8 +157,8 @@ module.exports = function doBrowserify (project, platformApi, pluginInfo

[GitHub] cordova-lib pull request #515: CB-12383 Fix cordova_plugins metadata definit...

2017-01-25 Thread daserge
Github user daserge commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/515#discussion_r97741534 --- Diff: cordova-lib/src/plugman/browserify.js --- @@ -157,8 +157,8 @@ module.exports = function doBrowserify (project, platformApi, pluginInfoProvider

[GitHub] cordova-lib pull request #515: CB-12383 Fix cordova_plugins metadata definit...

2017-01-25 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/515#discussion_r97738068 --- Diff: cordova-lib/src/plugman/browserify.js --- @@ -157,8 +157,8 @@ module.exports = function doBrowserify (project, platformApi, pluginInfo