[GitHub] cordova-plugin-battery-status pull request #42: CB-8929 Use PowerManager to ...

2016-11-08 Thread daserge
Github user daserge commented on a diff in the pull request: https://github.com/apache/cordova-plugin-battery-status/pull/42#discussion_r86942697 --- Diff: src/windows/BatteryProxy.js --- @@ -19,7 +19,38 @@ * */ -/* global WinJS, BatteryStatus */ +/* globa

[GitHub] cordova-plugin-battery-status issue #42: CB-8929 Use PowerManager to get bat...

2016-11-08 Thread daserge
Github user daserge commented on the issue: https://github.com/apache/cordova-plugin-battery-status/pull/42 Tested and it works good on all targets (Win 10 desktop, Win 10 Mobile, Windows Phone 8.1). LGTM apart the comments. --- If your project is set up for it, you can reply to

[GitHub] cordova-plugin-file-transfer issue #167: Fix log warning

2016-11-08 Thread lucatorella
Github user lucatorella commented on the issue: https://github.com/apache/cordova-plugin-file-transfer/pull/167 https://github.com/apache/cordova-plugin-file-transfer/pull/164 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] cordova-windows pull request #209: CB-12124: Make available device capabilit...

2016-11-08 Thread matrosov-nikita
GitHub user matrosov-nikita opened a pull request: https://github.com/apache/cordova-windows/pull/209 CB-12124: Make available device capabilities in package.windows10.appxmanifest ### What does this PR do? This PR makes available to use device capabilities in package

[GitHub] cordova-windows issue #209: CB-12124: Make available device capabilities in ...

2016-11-08 Thread codecov-io
Github user codecov-io commented on the issue: https://github.com/apache/cordova-windows/pull/209 ## [Current coverage](https://codecov.io/gh/apache/cordova-windows/pull/209?src=pr) is 74.64% (diff: 100%) > Merging [#209](https://codecov.io/gh/apache/cordova-windows/pull/209?src=

[GitHub] cordova-windows issue #209: CB-12124: Make available device capabilities in ...

2016-11-08 Thread daserge
Github user daserge commented on the issue: https://github.com/apache/cordova-windows/pull/209 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, o

[GitHub] cordova-plugin-inappbrowser issue #125: CB-9445 Improves executeScript callb...

2016-11-08 Thread wanjochan
Github user wanjochan commented on the issue: https://github.com/apache/cordova-plugin-inappbrowser/pull/125 failed at simulator (iphone6s+, 9.2) , the last alert() in the callback is not working --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] cordova-plugin-battery-status issue #42: CB-8929 Use PowerManager to get bat...

2016-11-08 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the issue: https://github.com/apache/cordova-plugin-battery-status/pull/42 @daserge, addressed your notes --- 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

[GitHub] cordova-plugin-battery-status issue #42: CB-8929 Use PowerManager to get bat...

2016-11-08 Thread daserge
Github user daserge commented on the issue: https://github.com/apache/cordova-plugin-battery-status/pull/42 @vladimir-kotikov LGTM, thanks! --- 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 thi

[GitHub] cordova-plugin-battery-status pull request #42: CB-8929 Use PowerManager to ...

2016-11-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-battery-status/pull/42 --- 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-plugin-inappbrowser issue #187: CB-11136: InAppBrowser fails to clos...

2016-11-08 Thread jonathanli2
Github user jonathanli2 commented on the issue: https://github.com/apache/cordova-plugin-inappbrowser/pull/187 Would it be better to add an if condition check, and only apply the change if WKWebView is used? so that the change will not affect UIWebView's old behavior by any chance.

[GitHub] cordova-plugin-battery-status issue #42: CB-8929 Use PowerManager to get bat...

2016-11-08 Thread cordova-qa
Github user cordova-qa commented on the issue: https://github.com/apache/cordova-plugin-battery-status/pull/42 Cordova CI Build has one or more failures. **Commit** - [Link](https://github.com/apache/cordova-plugin-battery-status/pull/42/commits/9f236357b0488e8aecad852539

[GitHub] cordova-ios pull request #274: CB-12118 - Cordova run ios does not automatic...

2016-11-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-ios/pull/274 --- 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 feature is

Nightly build #221 for cordova has succeeded!

2016-11-08 Thread Apache Jenkins Server
Nightly build #221 for cordova has succeeded! The latest nightly has been published and you can try it out with 'npm i -g cordova@nightly' For details check build console at https://builds.apache.org/job/cordova-nightly/221/consoleFull - Jenkins for Apache Cordova -

[GitHub] cordova-plugin-battery-status pull request #43: Update README.md

2016-11-08 Thread APMann
GitHub user APMann opened a pull request: https://github.com/apache/cordova-plugin-battery-status/pull/43 Update README.md The documentation is wrong, the functions return two objects instead of one. ### Platforms affected ### What does this PR d

[GitHub] cordova-plugin-battery-status issue #43: Update README.md

2016-11-08 Thread cordova-qa
Github user cordova-qa commented on the issue: https://github.com/apache/cordova-plugin-battery-status/pull/43 Cordova CI Build has one or more failures. **Commit** - [Link](https://github.com/apache/cordova-plugin-battery-status/pull/43/commits/faef22b293e0b0af2bf95d70af

[GitHub] cordova-windows pull request #209: CB-12124: Make available device capabilit...

2016-11-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-windows/pull/209 --- 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 #209: CB-12124: Make available device capabilities in ...

2016-11-08 Thread daserge
Github user daserge commented on the issue: https://github.com/apache/cordova-windows/pull/209 Merged, thanks! --- 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