[GitHub] cordova-plugin-media pull request #117: CB-11641 android: add 'message' fiel...

2016-09-26 Thread matrosov-nikita
GitHub user matrosov-nikita opened a pull request:

https://github.com/apache/cordova-plugin-media/pull/117

CB-11641 android: add 'message' field to media error



### Platforms affected
Android

### What does this PR do?
It was not possible before to get error message.This updated Android 
implementation to return error code with message similar to iOS.

### What testing has been done on this change?
Auto test

### Checklist
- [x] [ICLA](http://www.apache.org/licenses/icla.txt) has been signed and 
submitted to secret...@apache.org.
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [x] Added automated test coverage as appropriate for this change.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/matrosov-nikita/cordova-plugin-media 
CB-11641-check

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-media/pull/117.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #117


commit 9d0261c3ac9b7ccd1dc0d6873bd181f5121e6e68
Author: Nikita Matrosov 
Date:   2016-09-27T06:49:24Z

CB-11641 android: add 'message' field to media error

It was not possible before to get error message.This updated Android 
implementation to return error code with message similar to iOS.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Nightly build #158 for cordova has succeeded!

2016-09-26 Thread Apache Jenkins Server
Nightly build #158 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/158/consoleFull

-
Jenkins for Apache Cordova

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

[GitHub] cordova-docs pull request #643: CB-10929 changed OXS to macOS on filter butt...

2016-09-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-docs/pull/643


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-docs pull request #643: CB-10929 changed OXS to macOS on filter butt...

2016-09-26 Thread audreyso
GitHub user audreyso opened a pull request:

https://github.com/apache/cordova-docs/pull/643

CB-10929 changed OXS to macOS on filter button

CB-10929 changed OXS to macOS on filter button

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/audreyso/cordova-docs master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/643.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #643


commit 39edb391e78bd4d8f2eb6423df08955b7b43bf5f
Author: audreyso 
Date:   2016-09-26T23:38:51Z

CB-10929 added OXS filter button to plugin search and removed fire os 
filter button

commit 83538adb285122f4a3245cda498f48db41e50b3b
Author: audreyso 
Date:   2016-09-26T23:57:03Z

CB-10929 changed OXS to macOS on filter button




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-docs issue #642: CB-10929 added OXS filter button to plugin search a...

2016-09-26 Thread stevengill
Github user stevengill commented on the issue:

https://github.com/apache/cordova-docs/pull/642
  
Send a new pr. I already closed this one :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-docs issue #642: CB-10929 added OXS filter button to plugin search a...

2016-09-26 Thread audreyso
Github user audreyso commented on the issue:

https://github.com/apache/cordova-docs/pull/642
  
Sure! I can do that… that makes sense. Thanks for pointing that out.

From: Darryl Pogue 
Reply-To: apache/cordova-docs 
Date: Monday, September 26, 2016 at 4:46 PM
To: apache/cordova-docs 
Cc: auso , Author 
Subject: Re: [apache/cordova-docs] CB-10929 added OXS filter button to 
plugin search and removed fire os… (#642)


With 10.12, Apple rebranded their desktop OS to macOS. Maybe we want to say 
OSX / macOS?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on 
GitHub, 
or mute the 
thread.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-docs pull request #642: CB-10929 added OXS filter button to plugin s...

2016-09-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-docs/pull/642


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-docs issue #642: CB-10929 added OXS filter button to plugin search a...

2016-09-26 Thread dpogue
Github user dpogue commented on the issue:

https://github.com/apache/cordova-docs/pull/642
  
With 10.12, Apple rebranded their desktop OS to `macOS`. Maybe we want to 
say `OSX / macOS`?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-docs pull request #642: CB-10929 added OXS filter button to plugin s...

2016-09-26 Thread audreyso
GitHub user audreyso opened a pull request:

https://github.com/apache/cordova-docs/pull/642

CB-10929 added OXS filter button to plugin search and removed fire os…

CB-10929 added OXS filter button to plugin search and removed fire os 
filter button

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/audreyso/cordova-docs master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/642.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #642


commit 39edb391e78bd4d8f2eb6423df08955b7b43bf5f
Author: audreyso 
Date:   2016-09-26T23:38:51Z

CB-10929 added OXS filter button to plugin search and removed fire os 
filter button




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-lib issue #493: CB-11908 Handle edit-config in config.xml on prepare

2016-09-26 Thread stevengill
Github user stevengill commented on the issue:

https://github.com/apache/cordova-lib/pull/493
  
Travis and appveyor failures are likely due to missing 
https://github.com/apache/cordova-lib/pull/492


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

2016-09-26 Thread stevengill
Github user stevengill commented on the issue:

https://github.com/apache/cordova-lib/pull/492
  
JSHint is failing for you


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-android pull request #336: CB-11907: Upgrading the Gradle Plugins so...

2016-09-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-android/pull/336


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-android issue #336: CB-11907: Upgrading the Gradle Plugins so they w...

2016-09-26 Thread codecov-io
Github user codecov-io commented on the issue:

https://github.com/apache/cordova-android/pull/336
  
## [Current 
coverage](https://codecov.io/gh/apache/cordova-android/pull/336?src=pr) is 
35.15% (diff: 100%)
> Merging 
[#336](https://codecov.io/gh/apache/cordova-android/pull/336?src=pr) into 
[master](https://codecov.io/gh/apache/cordova-android/branch/master?src=pr) 
will not change coverage

```diff
@@ master   #336   diff @@
==
  Files12 12  
  Lines  1007   1007  
  Methods 198198  
  Messages  0  0  
  Branches168168  
==
  Hits354354  
  Misses  653653  
  Partials  0  0  
```

> Powered by [Codecov](https://codecov.io?src=pr). Last update 
[a4f6c8b...616ed79](https://codecov.io/gh/apache/cordova-android/compare/a4f6c8bf7bf5ba49855f31831662f7964b17f962...616ed79782dfc4978a08db51069713f5d09decaf?src=pr)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



RE: Hi!

2016-09-26 Thread Ryan J. Salva
Welcome, Audrey!



Ryan J. Salva  |  Principal PM Manager
Visual Studio Mobile Developer Tools
rsa...@microsoft.com
425 706 5270 office
206 612 5079 mobile

-Original Message-
From: Audrey So [mailto:a...@adobe.com] 
Sent: Monday, September 26, 2016 3:13 PM
To: dev@cordova.apache.org
Subject: Hi!

Hi! Just wanted to send an introductory e-mail! My name is Audrey and I’m the 
fall intern here in San Francisco – I’m very happy to learn from and work with 
the Cordova team! Feel free to message me!

Best,
Audrey


Hi!

2016-09-26 Thread Audrey So
Hi! Just wanted to send an introductory e-mail! My name is Audrey and I’m the 
fall intern here in San Francisco – I’m very happy to learn from and work with 
the Cordova team! Feel free to message me!

Best,
Audrey


[GitHub] cordova-android pull request #336: CB-11907: Upgrading the Gradle Plugins so...

2016-09-26 Thread infil00p
GitHub user infil00p opened a pull request:

https://github.com/apache/cordova-android/pull/336

CB-11907: Upgrading the Gradle Plugins so they work with Android Studio 2.2

This is a PR to run against our CI before I put these into master.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/infil00p/cordova-android master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-android/pull/336.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #336






---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-inappbrowser issue #187: CB-11136: InAppBrowser fails to clos...

2016-09-26 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-inappbrowser/pull/187
  
Cordova CI Build has completed successfully.

**Commit** - 
[Link](https://github.com/apache/cordova-plugin-inappbrowser/pull/187/commits/f3a8fbe1c0737138d4b0e1b358b1c2d6d5d2c16b)
**Dashboard** - 
[Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78/)

| Builder Name  | Console Output | Test Report | Device Logs  |
| :---: | :---:  |   :---: | :---:|
| [Windows 8.1 Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-store/artifact/)
 |
| [Windows 10  Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-10-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-10-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-10-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-10-store/artifact/)
 |
| [Windows 8.1 Phone]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-phone/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-phone/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-phone/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=windows-8.1-phone/artifact/)
 |
| [iOS]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=ios/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=ios/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=ios/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=ios/artifact/)
 |
| [Android]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=android/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=android/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=android/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-inappbrowser-pr/78//PLATFORM=android/artifact/)
 |
 



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Re: Google Play change to Android ACCESS_FINE_LOCATION permissions

2016-09-26 Thread Joe Bowser
edit-config does work on config.xml, so this is really a documentation bug
for the next release.

On Mon, Sep 26, 2016 at 2:50 PM, julio cesar sanchez  wrote:

> Right now the plugin have both permissions, fine and coarse. Fine is only
> needed for enableHighAccuracy true.
>
> Maybe once config-file is fully supported on the config.xml we might remove
> permissions from the plugins and document how to set them from the
> config.xml
>
> El lunes, 26 de septiembre de 2016, Steven Gill 
> escribió:
>
> > our geolocation plugin comes with ACCESS_FINE_LOCATION permission. A
> second
> > plugin might make sense.
> >
> > On Mon, Sep 26, 2016 at 11:45 AM, julio cesar sanchez <
> > jcesarmob...@gmail.com > wrote:
> >
> > > The problem with this is, not everybody will want to use precise
> > location,
> > > so they don't need gps hardware.
> > >
> > > Should we create another plugin with just ACCESS_COARSE_LOCATION
> > permission
> > > and no gps feature?
> > > Or should we find a way of installing it so you can choose the
> permission
> > > on install time and then document that for using enableHighAccuracy you
> > > have to use the ACCESS_FINE_LOCATION permission and gps feature?
> > >
> > >
> > >
> > > 2016-09-22 5:07 GMT+02:00 Steven Gill  > >:
> > >
> > > > I'll deal with it and start a release for it
> > > >
> > > > On Sep 21, 2016 7:37 PM, "Joe Bowser"  > > wrote:
> > > >
> > > > > Someone already did it.
> > > > >
> > > > > https://issues.apache.org/jira/browse/CB-11875
> > > > >
> > > > > But yeah, this is a newbie one, since it's literally one line in
> > > > > plugin.xml.  I'll do it tomorrow, but if someone wants an easy PR
> > done,
> > > > do
> > > > > it up.
> > > > >
> > > > > On Wed, Sep 21, 2016 at 4:55 PM, Steven Gill <
> stevengil...@gmail.com
> > >
> > > > > wrote:
> > > > >
> > > > > > Do it!
> > > > > >
> > > > > > On Tue, Sep 20, 2016 at 12:20 PM, Joe Bowser  > >
> > > > wrote:
> > > > > >
> > > > > > > FYI.  I'm going to create an issue to add this to the
> plugin.xml
> > > for
> > > > > > > geolocation later.
> > > > > > >
> > > > > > > -- Forwarded message --
> > > > > > > From: Google Play  >
> > > > > > > Date: Tue, Sep 20, 2016 at 11:40 AM
> > > > > > > Subject: Google Play change to Android ACCESS_FINE_LOCATION
> > > > permissions
> > > > > > > To: bows...@gmail.com 
> > > > > > >
> > > > > > >
> > > > > > > Hello Google Play Developer,
> > > > > > >
> > > > > > > We're writing to let you know that the apps listed at the end
> of
> > > this
> > > > > > email
> > > > > > > may be affected by an upcoming platform change.
> > > > > > >
> > > > > > > *Action required*: If your app requires GPS hardware to operate
> > > > > properly,
> > > > > > > you will need to explicitly add the "android.hardware.location.
> > > gps"
> > > > > > > uses-feature to your manifest.
> > > > > > >
> > > > > > > *What’s changing*
> > > > > > >
> > > > > > > We’re making a change on October 15th, 2016 that will affect
> apps
> > > > > > targeting
> > > > > > > API version 21 (Android 5.0, Lollipop) or higher that use
> > > > > > > ACCESS_FINE_LOCATION but don't explicitly have the
> > > > > > > "android.hardware.location.gps" uses-feature. Going forward,
> > these
> > > > > apps
> > > > > > > will be available to install on devices that don't have GPS
> > > hardware.
> > > > > In
> > > > > > > most cases this won't be an issue since Wi-Fi and Cell-ID based
> > > > > location
> > > > > > > provides high enough fidelity for the typical operation of
> these
> > > > apps.
> > > > > > > However, any apps that require GPS hardware, such as GPS
> > > navigators,
> > > > > > should
> > > > > > > explicitly add the "android.hardware.location.gps"
> uses-feature
> > to
> > > > > their
> > > > > > > manifest.
> > > > > > >
> > > > > > > If your app requires GPS to function properly and you do not
> > > include
> > > > > > > android.hardware.location.gps in your manifest declaration,
> your
> > > > users
> > > > > > may
> > > > > > > have a poor app experience.
> > > > > > >
> > > > > > > Also, if you’re using the fused location provider and wish to
> > > receive
> > > > > the
> > > > > > > most accurate location samples from GPS (i.e. with
> > > > > > PRIORITY_HIGH_ACCURACY),
> > > > > > > you must include the "android.hardware.location.gps" feature
> in
> > > your
> > > > > > app’s
> > > > > > > manifest to ensure that Google Play only distributes your app
> to
> > > > > devices
> > > > > > > with GPS sensors.
> > > > > > >
> > > > > > > You can read more about this change in the Android Developers
> > Help
> > > > > Center
> > > > > > >  > 2Sk1mB8v3V5DE2_
> > > > > > > HelAsLfkx0mUzuENWxtqMA6Ax7dkRwzYtv5u1mnNjWZye17r3zTTrs1z_
> > > > Q2AUnAygTF2M-
> > > > > > > deVbIrIsu2evapXjKKgv4ttr0ngMFag=>
> > > > > > > .
> > > > > > >
> > > > > > > Regards,
> > > > > > >
> > > > > > > The Google Play Team
> > > > > > >
> > > > > > > Up to 20 affected app(s) and ve

Re: Google Play change to Android ACCESS_FINE_LOCATION permissions

2016-09-26 Thread julio cesar sanchez
Right now the plugin have both permissions, fine and coarse. Fine is only
needed for enableHighAccuracy true.

Maybe once config-file is fully supported on the config.xml we might remove
permissions from the plugins and document how to set them from the
config.xml

El lunes, 26 de septiembre de 2016, Steven Gill 
escribió:

> our geolocation plugin comes with ACCESS_FINE_LOCATION permission. A second
> plugin might make sense.
>
> On Mon, Sep 26, 2016 at 11:45 AM, julio cesar sanchez <
> jcesarmob...@gmail.com > wrote:
>
> > The problem with this is, not everybody will want to use precise
> location,
> > so they don't need gps hardware.
> >
> > Should we create another plugin with just ACCESS_COARSE_LOCATION
> permission
> > and no gps feature?
> > Or should we find a way of installing it so you can choose the permission
> > on install time and then document that for using enableHighAccuracy you
> > have to use the ACCESS_FINE_LOCATION permission and gps feature?
> >
> >
> >
> > 2016-09-22 5:07 GMT+02:00 Steven Gill  >:
> >
> > > I'll deal with it and start a release for it
> > >
> > > On Sep 21, 2016 7:37 PM, "Joe Bowser"  > wrote:
> > >
> > > > Someone already did it.
> > > >
> > > > https://issues.apache.org/jira/browse/CB-11875
> > > >
> > > > But yeah, this is a newbie one, since it's literally one line in
> > > > plugin.xml.  I'll do it tomorrow, but if someone wants an easy PR
> done,
> > > do
> > > > it up.
> > > >
> > > > On Wed, Sep 21, 2016 at 4:55 PM, Steven Gill  >
> > > > wrote:
> > > >
> > > > > Do it!
> > > > >
> > > > > On Tue, Sep 20, 2016 at 12:20 PM, Joe Bowser  >
> > > wrote:
> > > > >
> > > > > > FYI.  I'm going to create an issue to add this to the plugin.xml
> > for
> > > > > > geolocation later.
> > > > > >
> > > > > > -- Forwarded message --
> > > > > > From: Google Play >
> > > > > > Date: Tue, Sep 20, 2016 at 11:40 AM
> > > > > > Subject: Google Play change to Android ACCESS_FINE_LOCATION
> > > permissions
> > > > > > To: bows...@gmail.com 
> > > > > >
> > > > > >
> > > > > > Hello Google Play Developer,
> > > > > >
> > > > > > We're writing to let you know that the apps listed at the end of
> > this
> > > > > email
> > > > > > may be affected by an upcoming platform change.
> > > > > >
> > > > > > *Action required*: If your app requires GPS hardware to operate
> > > > properly,
> > > > > > you will need to explicitly add the "android.hardware.location.
> > gps"
> > > > > > uses-feature to your manifest.
> > > > > >
> > > > > > *What’s changing*
> > > > > >
> > > > > > We’re making a change on October 15th, 2016 that will affect apps
> > > > > targeting
> > > > > > API version 21 (Android 5.0, Lollipop) or higher that use
> > > > > > ACCESS_FINE_LOCATION but don't explicitly have the
> > > > > > "android.hardware.location.gps" uses-feature. Going forward,
> these
> > > > apps
> > > > > > will be available to install on devices that don't have GPS
> > hardware.
> > > > In
> > > > > > most cases this won't be an issue since Wi-Fi and Cell-ID based
> > > > location
> > > > > > provides high enough fidelity for the typical operation of these
> > > apps.
> > > > > > However, any apps that require GPS hardware, such as GPS
> > navigators,
> > > > > should
> > > > > > explicitly add the "android.hardware.location.gps" uses-feature
> to
> > > > their
> > > > > > manifest.
> > > > > >
> > > > > > If your app requires GPS to function properly and you do not
> > include
> > > > > > android.hardware.location.gps in your manifest declaration, your
> > > users
> > > > > may
> > > > > > have a poor app experience.
> > > > > >
> > > > > > Also, if you’re using the fused location provider and wish to
> > receive
> > > > the
> > > > > > most accurate location samples from GPS (i.e. with
> > > > > PRIORITY_HIGH_ACCURACY),
> > > > > > you must include the "android.hardware.location.gps" feature in
> > your
> > > > > app’s
> > > > > > manifest to ensure that Google Play only distributes your app to
> > > > devices
> > > > > > with GPS sensors.
> > > > > >
> > > > > > You can read more about this change in the Android Developers
> Help
> > > > Center
> > > > > >  2Sk1mB8v3V5DE2_
> > > > > > HelAsLfkx0mUzuENWxtqMA6Ax7dkRwzYtv5u1mnNjWZye17r3zTTrs1z_
> > > Q2AUnAygTF2M-
> > > > > > deVbIrIsu2evapXjKKgv4ttr0ngMFag=>
> > > > > > .
> > > > > >
> > > > > > Regards,
> > > > > >
> > > > > > The Google Play Team
> > > > > >
> > > > > > Up to 20 affected app(s) and version(s) are listed below. If you
> > have
> > > > > more
> > > > > > than 20 apps in your catalogue, make sure to check each app for
> > this
> > > > > issue.
> > > > > >
> > > > > > ca.dontflyhere.app   18
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> 

[VOTE] cordova-plugin-geolocation@2.4.0 Release

2016-09-26 Thread Steven Gill
Please review and vote on the release of this cordova-plugin-geolocation release
by replying to this email (and keep discussion on the DISCUSS thread)

Release issue: https://issues.apache.org/jira/browse/CB-11904

The plugin has been published to
dist/dev:https://dist.apache.org/repos/dist/dev/cordova/CB-11904/

The package was published from its corresponding git tag:
cordova-plugin-geolocation: 2.4.0 (111c747823)

Upon a successful vote I will upload the archives to dist/, upload
them to npm, and post the corresponding blog post.

Voting guidelines:
https://github.com/apache/cordova-coho/blob/master/docs/release-voting.md
*NEW:*
How to vote on a plugins release at
https://github.com/apache/cordova-coho/blob/master/docs/plugins-release-process.md#voting

Voting will go on for a minimum of 48 hours.

I vote +1:
* Ran coho audit-license-headers over the relevant repos
* Ran coho check-license to ensure all dependencies and
subdependencies have Apache-compatible licenses
* Ensured continuous build was green when repos were tagged


[GitHub] cordova-plugin-inappbrowser issue #187: CB-11136: InAppBrowser fails to clos...

2016-09-26 Thread manucorporat
Github user manucorporat commented on the issue:

https://github.com/apache/cordova-plugin-inappbrowser/pull/187
  
@shazron done!
renamed to tmpWindow and tmpController


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Re: Google Play change to Android ACCESS_FINE_LOCATION permissions

2016-09-26 Thread Steven Gill
our geolocation plugin comes with ACCESS_FINE_LOCATION permission. A second
plugin might make sense.

On Mon, Sep 26, 2016 at 11:45 AM, julio cesar sanchez <
jcesarmob...@gmail.com> wrote:

> The problem with this is, not everybody will want to use precise location,
> so they don't need gps hardware.
>
> Should we create another plugin with just ACCESS_COARSE_LOCATION permission
> and no gps feature?
> Or should we find a way of installing it so you can choose the permission
> on install time and then document that for using enableHighAccuracy you
> have to use the ACCESS_FINE_LOCATION permission and gps feature?
>
>
>
> 2016-09-22 5:07 GMT+02:00 Steven Gill :
>
> > I'll deal with it and start a release for it
> >
> > On Sep 21, 2016 7:37 PM, "Joe Bowser"  wrote:
> >
> > > Someone already did it.
> > >
> > > https://issues.apache.org/jira/browse/CB-11875
> > >
> > > But yeah, this is a newbie one, since it's literally one line in
> > > plugin.xml.  I'll do it tomorrow, but if someone wants an easy PR done,
> > do
> > > it up.
> > >
> > > On Wed, Sep 21, 2016 at 4:55 PM, Steven Gill 
> > > wrote:
> > >
> > > > Do it!
> > > >
> > > > On Tue, Sep 20, 2016 at 12:20 PM, Joe Bowser 
> > wrote:
> > > >
> > > > > FYI.  I'm going to create an issue to add this to the plugin.xml
> for
> > > > > geolocation later.
> > > > >
> > > > > -- Forwarded message --
> > > > > From: Google Play 
> > > > > Date: Tue, Sep 20, 2016 at 11:40 AM
> > > > > Subject: Google Play change to Android ACCESS_FINE_LOCATION
> > permissions
> > > > > To: bows...@gmail.com
> > > > >
> > > > >
> > > > > Hello Google Play Developer,
> > > > >
> > > > > We're writing to let you know that the apps listed at the end of
> this
> > > > email
> > > > > may be affected by an upcoming platform change.
> > > > >
> > > > > *Action required*: If your app requires GPS hardware to operate
> > > properly,
> > > > > you will need to explicitly add the "android.hardware.location.
> gps"
> > > > > uses-feature to your manifest.
> > > > >
> > > > > *What’s changing*
> > > > >
> > > > > We’re making a change on October 15th, 2016 that will affect apps
> > > > targeting
> > > > > API version 21 (Android 5.0, Lollipop) or higher that use
> > > > > ACCESS_FINE_LOCATION but don't explicitly have the
> > > > > "android.hardware.location.gps" uses-feature. Going forward, these
> > > apps
> > > > > will be available to install on devices that don't have GPS
> hardware.
> > > In
> > > > > most cases this won't be an issue since Wi-Fi and Cell-ID based
> > > location
> > > > > provides high enough fidelity for the typical operation of these
> > apps.
> > > > > However, any apps that require GPS hardware, such as GPS
> navigators,
> > > > should
> > > > > explicitly add the "android.hardware.location.gps" uses-feature to
> > > their
> > > > > manifest.
> > > > >
> > > > > If your app requires GPS to function properly and you do not
> include
> > > > > android.hardware.location.gps in your manifest declaration, your
> > users
> > > > may
> > > > > have a poor app experience.
> > > > >
> > > > > Also, if you’re using the fused location provider and wish to
> receive
> > > the
> > > > > most accurate location samples from GPS (i.e. with
> > > > PRIORITY_HIGH_ACCURACY),
> > > > > you must include the "android.hardware.location.gps" feature in
> your
> > > > app’s
> > > > > manifest to ensure that Google Play only distributes your app to
> > > devices
> > > > > with GPS sensors.
> > > > >
> > > > > You can read more about this change in the Android Developers Help
> > > Center
> > > > >  > > > > HelAsLfkx0mUzuENWxtqMA6Ax7dkRwzYtv5u1mnNjWZye17r3zTTrs1z_
> > Q2AUnAygTF2M-
> > > > > deVbIrIsu2evapXjKKgv4ttr0ngMFag=>
> > > > > .
> > > > >
> > > > > Regards,
> > > > >
> > > > > The Google Play Team
> > > > >
> > > > > Up to 20 affected app(s) and version(s) are listed below. If you
> have
> > > > more
> > > > > than 20 apps in your catalogue, make sure to check each app for
> this
> > > > issue.
> > > > >
> > > > > ca.dontflyhere.app   18
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > ©2016 Google Inc. 1600 Amphitheatre Parkway, Mountain View, CA
> 94043
> > > > >
> > > > > Email preferences: You have received this mandatory email service
> > > > > announcement to update you about important changes to your Google
> > Play
> > > > > Developer account.
> > > > >
> > > >
> > >
> >
>


[GitHub] cordova-lib pull request #493: CB-11908 Handle edit-config in config.xml on ...

2016-09-26 Thread ktop
GitHub user ktop opened a pull request:

https://github.com/apache/cordova-lib/pull/493

CB-11908 Handle edit-config in config.xml on prepare

Needs this PR: https://github.com/apache/cordova-lib/pull/492

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ktop/cordova-lib libEditConfig

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/493.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #493


commit 34b50acee4fa9eb27d7225a3e4497c7917570667
Author: ktop 
Date:   2016-09-26T21:10:15Z

CB-11908 Handle edit-config in config.xml on prepare




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-lib pull request #492: CB-11908 Add edit-config to config.xml

2016-09-26 Thread ktop
GitHub user ktop opened a pull request:

https://github.com/apache/cordova-lib/pull/492

CB-11908 Add edit-config to config.xml

Handles edit-config for config.xml. 

Fixed up the conflict checking so that config.xml will take precedence over 
plugin.xml changes. 
Added a remove mode to config.xml only, explained more below. 

To test out these changes, remember to copy it to the platform's 
cordova-common. 

A couple things to discuss/clarify:
1. These changes will make config.xml changes take precedence over 
plugin.xml changes even if --force is used. Is this alright? Or should --force 
actually force add the plugin anyways? 

2. There's no real way to "undo" edit-config changes in config.xml. As a 
fix, I've added a "remove" mode that will remove the attributes specified. Do 
you think this is useful to have? There's no such thing as restoring/reverting 
the xml elements to what they were before. What was initially the old list of 
attributes from before the edit-config change will get overwritten on every 
prepare. 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ktop/cordova-lib editconfigxml

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/492.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #492


commit d0ce4c506d2f7f24bda145e21f9b9af1543b1ecd
Author: ktop 
Date:   2016-09-26T21:07:09Z

CB-11908 Add edit-config to config.xml




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-geolocation pull request #79: [CB-11617] (ubuntu) confusion b...

2016-09-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-geolocation/pull/79


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Code/Doc review for CB-9762 & CB-11829/11830

2016-09-26 Thread Kerri Shotts
I’ve added the storyboard support as described in CB-9762 to the splashscreen 
plugin in CB-11829 and have finished the docs for the feature as well. It’d be 
great to have a couple eyeballs take a quick look at the following PRs and 
seeing if anything jumps out. :-) 

Core support: https://github.com/apache/cordova-ios/pull/250/files
Plugin changes & doc: 
https://github.com/apache/cordova-plugin-splashscreen/pull/114/files

CI is happy, and all tests pass, although I should note that no new tests have 
been introduced. One can test the visuals manually using the splashscreen 
plugin’s manual tests on real devices (see quirks section of docs).

Note: I’ve put the docs for the entire feature in the splash screen docs, since 
that’s currently the only place we have any documentation relating to splash 
screens. If we want to put this documentation elsewhere or split it out (it’s 
long!), let me know.

~Kerri
-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-splashscreen issue #114: CB-11829: (iOS) Support for CB-9762;...

2016-09-26 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-splashscreen/pull/114
  
Cordova CI Build has completed successfully.

**Commit** - 
[Link](https://github.com/apache/cordova-plugin-splashscreen/pull/114/commits/83a7ca0b02b6217ced8ca7116fd34b449fb06fc1)
**Dashboard** - 
[Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27/)

| Builder Name  | Console Output | Test Report | Device Logs  |
| :---: | :---:  |   :---: | :---:|
| [Windows 8.1 Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-store/artifact/)
 |
| [Windows 10  Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-10-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-10-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-10-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-10-store/artifact/)
 |
| [Windows 8.1 Phone]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-phone/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-phone/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-phone/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=windows-8.1-phone/artifact/)
 |
| [iOS]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=ios/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=ios/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=ios/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=ios/artifact/)
 |
| [Android]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=android/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=android/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=android/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-splashscreen-pr/27//PLATFORM=android/artifact/)
 |
 



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Re: [PROPOSAL] Deprecation of node.js 0.x and upgrade to a minimum usage of node.js 6.x in Apache Cordova

2016-09-26 Thread Shazron
Updated with timeline graphic for the two deprecation periods proposed:
https://github.com/cordova/cordova-discuss/pull/52#issuecomment-249656953

I'll update the doc once the timeline has consensus.

On Fri, Sep 23, 2016 at 12:48 PM, Shazron  wrote:

> Surfacing this again: https://github.com/cordova/cordova-discuss/pull/52
>
> I need more feedback from stakeholders (especially enterprise users).
> There has been pushback to not make the leap to node 6, but move to node 4
> instead after 0.x support is deprecated.
>
>
>
> On Tue, Sep 20, 2016 at 5:04 PM, Shazron  wrote:
>
>> See: https://github.com/cordova/cordova-discuss/pull/52
>>
>> Please discuss and comment directly on the PR. Finalized proposal will be
>> pasted back into the list.
>>
>
>


Re: Google Play change to Android ACCESS_FINE_LOCATION permissions

2016-09-26 Thread julio cesar sanchez
The problem with this is, not everybody will want to use precise location,
so they don't need gps hardware.

Should we create another plugin with just ACCESS_COARSE_LOCATION permission
and no gps feature?
Or should we find a way of installing it so you can choose the permission
on install time and then document that for using enableHighAccuracy you
have to use the ACCESS_FINE_LOCATION permission and gps feature?



2016-09-22 5:07 GMT+02:00 Steven Gill :

> I'll deal with it and start a release for it
>
> On Sep 21, 2016 7:37 PM, "Joe Bowser"  wrote:
>
> > Someone already did it.
> >
> > https://issues.apache.org/jira/browse/CB-11875
> >
> > But yeah, this is a newbie one, since it's literally one line in
> > plugin.xml.  I'll do it tomorrow, but if someone wants an easy PR done,
> do
> > it up.
> >
> > On Wed, Sep 21, 2016 at 4:55 PM, Steven Gill 
> > wrote:
> >
> > > Do it!
> > >
> > > On Tue, Sep 20, 2016 at 12:20 PM, Joe Bowser 
> wrote:
> > >
> > > > FYI.  I'm going to create an issue to add this to the plugin.xml for
> > > > geolocation later.
> > > >
> > > > -- Forwarded message --
> > > > From: Google Play 
> > > > Date: Tue, Sep 20, 2016 at 11:40 AM
> > > > Subject: Google Play change to Android ACCESS_FINE_LOCATION
> permissions
> > > > To: bows...@gmail.com
> > > >
> > > >
> > > > Hello Google Play Developer,
> > > >
> > > > We're writing to let you know that the apps listed at the end of this
> > > email
> > > > may be affected by an upcoming platform change.
> > > >
> > > > *Action required*: If your app requires GPS hardware to operate
> > properly,
> > > > you will need to explicitly add the "android.hardware.location.gps"
> > > > uses-feature to your manifest.
> > > >
> > > > *What’s changing*
> > > >
> > > > We’re making a change on October 15th, 2016 that will affect apps
> > > targeting
> > > > API version 21 (Android 5.0, Lollipop) or higher that use
> > > > ACCESS_FINE_LOCATION but don't explicitly have the
> > > > "android.hardware.location.gps" uses-feature. Going forward, these
> > apps
> > > > will be available to install on devices that don't have GPS hardware.
> > In
> > > > most cases this won't be an issue since Wi-Fi and Cell-ID based
> > location
> > > > provides high enough fidelity for the typical operation of these
> apps.
> > > > However, any apps that require GPS hardware, such as GPS navigators,
> > > should
> > > > explicitly add the "android.hardware.location.gps" uses-feature to
> > their
> > > > manifest.
> > > >
> > > > If your app requires GPS to function properly and you do not include
> > > > android.hardware.location.gps in your manifest declaration, your
> users
> > > may
> > > > have a poor app experience.
> > > >
> > > > Also, if you’re using the fused location provider and wish to receive
> > the
> > > > most accurate location samples from GPS (i.e. with
> > > PRIORITY_HIGH_ACCURACY),
> > > > you must include the "android.hardware.location.gps" feature in your
> > > app’s
> > > > manifest to ensure that Google Play only distributes your app to
> > devices
> > > > with GPS sensors.
> > > >
> > > > You can read more about this change in the Android Developers Help
> > Center
> > > >  > > > HelAsLfkx0mUzuENWxtqMA6Ax7dkRwzYtv5u1mnNjWZye17r3zTTrs1z_
> Q2AUnAygTF2M-
> > > > deVbIrIsu2evapXjKKgv4ttr0ngMFag=>
> > > > .
> > > >
> > > > Regards,
> > > >
> > > > The Google Play Team
> > > >
> > > > Up to 20 affected app(s) and version(s) are listed below. If you have
> > > more
> > > > than 20 apps in your catalogue, make sure to check each app for this
> > > issue.
> > > >
> > > > ca.dontflyhere.app   18
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > ©2016 Google Inc. 1600 Amphitheatre Parkway, Mountain View, CA 94043
> > > >
> > > > Email preferences: You have received this mandatory email service
> > > > announcement to update you about important changes to your Google
> Play
> > > > Developer account.
> > > >
> > >
> >
>


Re: [Discuss] cordova-plugin-geolocation release

2016-09-26 Thread Steven Gill
https://issues.apache.org/jira/browse/CB-11904

On Fri, Sep 23, 2016 at 11:05 AM, Joe Bowser  wrote:

> On Fri, Sep 23, 2016 at 7:13 AM, Pridham, Marcus 
> wrote:
>
> > This change might cause issues if you have an Android application that
> > perhaps you want to use GPS on the phone but also want to allow to run on
> > Chromebooks?  I got an email recently from Google about the
> > android.hardware.location.gps permission:
> >
> >
> > “In preparation for the release of Chromebooks support for the Google
> Play
> > Store and Android apps, we've been proactively testing the most key and
> > popular applications in Google Play, including *** for compatibility.
> >
> > We wanted to let you know, currently your app isn't compatible on Chrome
> > OS devices due to manifest entry android.hardware.location.gps. If you
> want
> > to make it compatible, Please read our guide on App Manifest
> Compatibility
> > for more details on how to ensure your app is compatible.”
> >
> >
> > I don’t have a Chromebook to try out what happens with this plugin there…
> > I think for the Chromebook case you would need required=false and the
> > geolocation would need to fail gracefully or the application code would
> > need to check the useragent and not call the plugin?
> >
> >  > android:required="false" />
> >
> > Has anyone tried a Cordova application on a Chromebook?
> >
> >
> Yes, and Cordova Apps not supported by the Play Store currently.  Besides
> that, I think running a Cordova-Android app on a Chromebook is missing the
> point.
>
>
> > On 9/23/16, 3:14 AM, "julio cesar sanchez" 
> wrote:
> >
> > +1
> >
> >
> > 2016-09-23 9:09 GMT+02:00 Shazron :
> >
> > > +1
> > >
> > > On Thu, Sep 22, 2016 at 5:02 PM, Steven Gill <
> stevengil...@gmail.com
> > >
> > > wrote:
> > >
> > > > Thoughts? Just want to get the fix for
> > > > https://issues.apache.org/jira/browse/CB-11875 out.
> > > >
> > > > -Steve
> > > >
> > >
> >
> >
> >
>


[GitHub] cordova-plugin-contacts issue #136: CB-11028 android: Allow to set custom la...

2016-09-26 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-contacts/pull/136
  
Cordova CI Build has completed successfully.

**Commit** - 
[Link](https://github.com/apache/cordova-plugin-contacts/pull/136/commits/200f1fdd9d69bdad2c3d4c315f6c4bc40159ae6e)
**Dashboard** - 
[Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39/)

| Builder Name  | Console Output | Test Report | Device Logs  |
| :---: | :---:  |   :---: | :---:|
| [Windows 8.1 Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-store/artifact/)
 |
| [Windows 10  Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-10-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-10-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-10-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-10-store/artifact/)
 |
| [Windows 8.1 Phone]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-phone/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-phone/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-phone/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=windows-8.1-phone/artifact/)
 |
| [iOS]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=ios/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=ios/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=ios/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=ios/artifact/)
 |
| [Android]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=android/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=android/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=android/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/39//PLATFORM=android/artifact/)
 |
 



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-contacts issue #135: CB-11350 android: retrieve displayName f...

2016-09-26 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-contacts/pull/135
  
Cordova CI Build has completed successfully.

**Commit** - 
[Link](https://github.com/apache/cordova-plugin-contacts/pull/135/commits/1940b1688e217acd5b40d8afcacef7a2e326409c)
**Dashboard** - 
[Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38/)

| Builder Name  | Console Output | Test Report | Device Logs  |
| :---: | :---:  |   :---: | :---:|
| [Windows 8.1 Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-store/artifact/)
 |
| [Windows 10  Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-10-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-10-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-10-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-10-store/artifact/)
 |
| [Windows 8.1 Phone]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-phone/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-phone/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-phone/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=windows-8.1-phone/artifact/)
 |
| [iOS]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=ios/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=ios/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=ios/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=ios/artifact/)
 |
| [Android]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=android/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=android/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=android/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-contacts-pr/38//PLATFORM=android/artifact/)
 |
 



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-contacts pull request #136: CB-11028 android: Allow to set cu...

2016-09-26 Thread matrosovN
GitHub user matrosovN opened a pull request:

https://github.com/apache/cordova-plugin-contacts/pull/136

CB-11028 android: Allow to set custom labels for contacts' fields



### Platforms affected
Android

### What does this PR do?
It was not possible before to retrieve the value of custom field's label 
(e.g. 'My phone'). The returned contact was always containing  for such fields. 
This updates Android implementation to return custom label's value rather than 
custom similar to iOS.

### What testing has been done on this change?
Auto test

### Checklist
- [x] [ICLA](http://www.apache.org/licenses/icla.txt) has been signed and 
submitted to secret...@apache.org.
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [x] Added automated test coverage as appropriate for this change.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/matrosovN/cordova-plugin-contacts CB-11028

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-contacts/pull/136.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #136


commit 200f1fdd9d69bdad2c3d4c315f6c4bc40159ae6e
Author: Nikita Matrosov 
Date:   2016-09-26T09:22:13Z

CB-11028 android: Allow to set custom labels for contacts' fields

It was not possible before to retrieve the value of custom field's label 
(e.g. 'My phone'). The returned contact was always containing  for such fields. 
This updates Android implementation to return custom label's value rather than 
custom similar to iOS.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-contacts pull request #135: CB-11350 android: retrieve displa...

2016-09-26 Thread matrosovN
GitHub user matrosovN opened a pull request:

https://github.com/apache/cordova-plugin-contacts/pull/135

CB-11350 android: retrieve displayName for contact when specified in 
desiredFields



### Platforms affected
Android

### What does this PR do?
DisplayName wasn't being retrieved because of wrong condition in 
populateContactArray. This PR allows to retrieve displayName for contact when 
specified in desiredFields.

### What testing has been done on this change?
Auto test

### Checklist
- [x] [ICLA](http://www.apache.org/licenses/icla.txt) has been signed and 
submitted to secret...@apache.org.
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [x] Added automated test coverage as appropriate for this change.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/matrosovN/cordova-plugin-contacts CB-11350

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-contacts/pull/135.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #135


commit 1940b1688e217acd5b40d8afcacef7a2e326409c
Author: Nikita Matrosov 
Date:   2016-09-26T08:42:56Z

CB-11350 android: retrieve displayName for contact when specified in 
desiredFields

displayName wasn't being retrieved because of wrong condition in 
populateContactArray




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org