[GitHub] cordova-windows pull request: CB-9335: Windows quality-of-life imp...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-windows/pull/98 --- 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

[GitHub] cordova-lib pull request: CB-9335: Windows quality-of-life improve...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-lib/pull/257 --- 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

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-lib/pull/259#issuecomment-121459962 Thanks @giuseppelt. Saw your ICLA came through. Change is merged. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-lib/pull/259 --- 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

[GitHub] cordova-windows pull request: CB-8674 - activationContext

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-windows/pull/97 --- 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

[GitHub] cordova-windows pull request: CB-8674 - activationContext

2015-07-14 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-windows/pull/97#issuecomment-121430365 This is merged and should close soon. The mirrors are slow today. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cordova-js pull request: CB-8972 Added common 'activated' channel

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-js/pull/119 --- 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

[GitHub] cordova-js pull request: CB-9368 remove old debug build steps

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-js/pull/120 --- 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

[GitHub] cordova-js pull request: CB-9368 remove old debug build steps

2015-07-14 Thread purplecabbage
GitHub user purplecabbage opened a pull request: https://github.com/apache/cordova-js/pull/120 CB-9368 remove old debug build steps You can merge this pull request into a Git repository by running: $ git pull https://github.com/purplecabbage/cordova-js CB-9368 Alternatively y

[Android] Working with M - An update

2015-07-14 Thread Joe Bowser
So, since Cordova-Android wasn't completely killed off by Google at the last Google IO like I thought it would be, we're going to have to make sure it's compatible with Android M (Marshmellow? Marzipan?). The good news is that this only affects the following plugins: MediaRecorder Contacts File F

[GitHub] cordova-js pull request: CB-8972 Added common 'activated' channel

2015-07-14 Thread purplecabbage
GitHub user purplecabbage opened a pull request: https://github.com/apache/cordova-js/pull/119 CB-8972 Added common 'activated' channel You can merge this pull request into a Git repository by running: $ git pull https://github.com/purplecabbage/cordova-js CB-8972 Alternative

[GitHub] cordova-plugin-camera pull request: Correct image resolution selec...

2015-07-14 Thread muratsu
Github user muratsu commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/56#issuecomment-121375282 @kiwinewt thanks for the contribution. The right approach here is to take into account the max num of pixels instead of a certain height or width limit.

[GitHub] cordova-plugin-contacts pull request: custom label for phoneNumber

2015-07-14 Thread typevitor
Github user typevitor commented on the pull request: https://github.com/apache/cordova-plugin-contacts/pull/67#issuecomment-121346507 Sorry, my mistake the removed hasPhoneNumber. I'll start a discussion then. --- If your project is set up for it, you can reply to this email and

[GitHub] cordova-registry-web pull request: CB-9209 add link from old page ...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-registry-web/pull/17 --- 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 fe

[GitHub] cordova-plugin-camera pull request: CB-6190 - iOS camera plugin ig...

2015-07-14 Thread jcesarmobile
GitHub user jcesarmobile opened a pull request: https://github.com/apache/cordova-plugin-camera/pull/108 CB-6190 - iOS camera plugin ignores quality parameter CB-6190 - iOS camera plugin ignores quality parameter in some circunstances Added a check to not downscale if quality

[GitHub] cordova-js pull request: CB-9366 Log error.stack as it helps with ...

2015-07-14 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-js/pull/118#issuecomment-121336324 I would leave this to userland. ```window.onerror = function(err,filename,lineNumber) { ... }``` Also, on iOS this is going to look like :

[GitHub] cordova-js pull request: CB-9366 Log error.stack as it helps with ...

2015-07-14 Thread nikhilkh
GitHub user nikhilkh opened a pull request: https://github.com/apache/cordova-js/pull/118 CB-9366 Log error.stack as it helps with diagnosing the source of the… … error You can merge this pull request into a Git repository by running: $ git pull https://github.com/MSOpenTec

[GitHub] cordova-windows pull request: CB-9239 Fixes issue with windows pre...

2015-07-14 Thread robpaveza
Github user robpaveza commented on the pull request: https://github.com/apache/cordova-windows/pull/99#issuecomment-121325599 I like the change overall but I need to still merge it with my latest to make sure it functionally works as expected. --- If your project is set up for it,

[GitHub] cordova-plugin-media pull request: CB-9238 Media plugin cannot rec...

2015-07-14 Thread robpaveza
Github user robpaveza commented on the pull request: https://github.com/apache/cordova-plugin-media/pull/58#issuecomment-121324570 Pending my other comments, 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

[GitHub] cordova-plugin-contacts pull request: custom label for phoneNumber

2015-07-14 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-contacts/pull/67#issuecomment-121323697 Um, no. You just removed the hasPhoneNumber functionality which was just added a couple weeks ago? You need to consider more than just Andr

[GitHub] cordova-windows pull request: CB-9239 Fixes issue with windows pre...

2015-07-14 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-windows/pull/99#issuecomment-121308567 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

[GitHub] cordova-plugin-contacts pull request: custom label for phoneNumber

2015-07-14 Thread typevitor
GitHub user typevitor opened a pull request: https://github.com/apache/cordova-plugin-contacts/pull/67 custom label for phoneNumber writing the possibility for adding custom label for phoneNumbers. You can merge this pull request into a Git repository by running: $ git pull htt

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread giuseppelt
Github user giuseppelt commented on the pull request: https://github.com/apache/cordova-lib/pull/259#issuecomment-121279489 I made the changes to use the promise. I signed the icla and mailed to secretary at apache. Do I have to do further actions? --- If your project is set up f

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/259#discussion_r34565570 --- Diff: cordova-lib/src/cordova/prepare.js --- @@ -106,28 +106,29 @@ function prepare(options) { if (options.browserify) {

[GitHub] cordova-windows pull request: CB-9239 Fixes issue with windows pre...

2015-07-14 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-windows/pull/99#issuecomment-121214310 @nikhilkh, i've added tests + Travis config. See https://github.com/vladimir-kotikov/cordova-windows/pull/1 for build status. Not sure about adding e2e

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread giuseppelt
Github user giuseppelt commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/259#discussion_r34558047 --- Diff: cordova-lib/src/cordova/prepare.js --- @@ -106,28 +106,29 @@ function prepare(options) { if (options.browserify) {

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/259#discussion_r34557772 --- Diff: cordova-lib/src/cordova/prepare.js --- @@ -106,28 +106,29 @@ function prepare(options) { if (options.browserify) {

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/259#discussion_r34557461 --- Diff: cordova-lib/src/cordova/prepare.js --- @@ -106,28 +106,29 @@ function prepare(options) { if (options.browserify) {

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-lib/pull/259#issuecomment-121200391 Thanks @giuseppelt. Looks good to me apart from the one stylistic comment. @stevengill does this look right to you? --- If your project is set up for it, you can

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/259#discussion_r34556418 --- Diff: cordova-lib/src/cordova/prepare.js --- @@ -106,28 +106,29 @@ function prepare(options) { if (options.browserify) {

[GitHub] cordova-windows pull request: Cb 9239

2015-07-14 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-windows/pull/100 Cb 9239 You can merge this pull request into a Git repository by running: $ git pull https://github.com/vladimir-kotikov/cordova-windows CB-9239 Alternatively you can review

[GitHub] cordova-windows pull request: Cb 9239

2015-07-14 Thread vladimir-kotikov
Github user vladimir-kotikov closed the pull request at: https://github.com/apache/cordova-windows/pull/100 --- 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-plugin-media pull request: iOS: Add support for playing no...

2015-07-14 Thread foxgeere
Github user foxgeere commented on the pull request: https://github.com/apache/cordova-plugin-media/pull/39#issuecomment-121192973 Is this going to be merged in any time soon? Really need to be able to play sound effects in app without it stopping system music in its tracks --- If yo

[GitHub] cordova-lib pull request: Fix prepare to wait the promise from plu...

2015-07-14 Thread giuseppelt
GitHub user giuseppelt opened a pull request: https://github.com/apache/cordova-lib/pull/259 Fix prepare to wait the promise from plugman prepare. This fixes the issue where after_prepare hook is fired before the prepare process ends. You can merge this pull request into a Git repo