Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-lib/pull/216#issuecomment-101574980
This should probably also use __.uniq Instead of the double loop, or
__.union
---
If your project is set up for it, you can reply to this email and have
Any chance this PR https://github.com/apache/cordova-plugin-camera/pull/83
https://github.com/apache/cordova-plugin-camera/pull/83 could get merged in?
regards Serge
a.k.a: thehuijb
signature.asc
Description: Message signed with OpenPGP using GPGMail
Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-lib/pull/216#issuecomment-101572757
+1
'concat does not alter this or any of the arrays ... '
Github user Gillardo commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/77#issuecomment-101711464
Found an issue though that if you zoom in, then the buttons disappear and
you cannot take the photo
---
If your project is set up for it, you can reply
Ah interesting. Where is the default config.xml actually defined? I
assume it is baked in and a user can't change the default used?
On Tue, May 12, 2015 at 11:02 AM, Andrew Grieve agri...@chromium.org wrote:
if you use --copy-from and the template doesn't already have a config.xml,
then the
It's here:
cordova-lib/cordova-lib/node_modules/cordova-app-hello-world/config.xml
A use can change the defaults by providing a config.xml within their own
template (--copy-from target)
On Wed, May 13, 2015 at 10:51 AM, Raymond Camden raymondcam...@gmail.com
wrote:
Ah interesting. Where is the
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-windows/pull/76
---
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
Github user muratsu commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30262108
--- Diff: src/windows/CameraProxy.js ---
@@ -337,28 +336,48 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args) {
Github user muratsu commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30262317
--- Diff: src/windows/CameraProxy.js ---
@@ -438,6 +457,61 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args) {
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-contacts/pull/62
---
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
Good idea. Maybe also file an issue under the 'cordova-registry-web' component
On Wed, May 13, 2015 at 12:42 PM, Josh Soref jso...@blackberry.com wrote:
I think it'd be helpful if plugins.cordova.io included a banner informing
people that it's going away.
Preferably w/ a link to something
Github user muratsu closed the pull request at:
https://github.com/apache/cordova-plugin-camera/pull/94
---
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
Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-windows/pull/76#issuecomment-101816672
Looks good to me too! +1 to merge.
I am not yet completely setup to test win10, so I am taking some things on
faith, but looking over the code it all
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-app-hello-world/pull/10#issuecomment-101816514
Is this required for development time experience or every app needs access
to localhost port 8472 to run on the phone?
---
If your project is set up
Github user jsoref commented on the pull request:
https://github.com/apache/cordova-app-hello-world/pull/10#issuecomment-101817138
It's required for runtime if an app (running on BlackBerry 10) has any
plugins. The app doesn't actually talk to port 8472, it's just magically
Rob, I am looking through it now and will get back to you later today.
I'll start a new thread on it too.
Cheers,
Jesse
@purplecabbage
risingj.com
On Tue, May 12, 2015 at 1:59 PM, Rob Paveza rob.pav...@microsoft.com
wrote:
Jesse, where have we landed on the issue of the local-notifications
https://github.com/apache/cordova-app-hello-world/pull/10#issuecomment-10142
9782
Problem:
As is, today, Cordova projects created w/ the default cordova hello-world
template which have any plugins won't work on BlackBerry 10 at-all
My fix is to add a CSP whitelist for the BlackBerry 10 magic
I think -dev is working fine and doing what it's supposed to do. The logic
of -label is subtle:
var sem = require('semver')
undefined
sem.satisfies('4.0.0-dev', '=4.0.0')
false
sem.satisfies('4.0.0-dev', '=4.0')
true
So, the engine check should just to =4.0, or =4.0.0-dev
On Wed, May 13,
Pushed a 5.0.0 for russian docs, updated edge docs for others
On Mon, May 11, 2015 at 10:52 AM, Steven Gill stevengil...@gmail.com
wrote:
Thanks Andrey,
I'll regenerate 5.0.0 and push an update later today.
-Steve
On Mon, May 11, 2015 at 6:53 AM, Andrey Kurdumov kant2...@googlemail.com
I think it'd be helpful if plugins.cordova.io included a banner informing
people that it's going away.
Preferably w/ a link to something explaining the transition plan+schedule.
smime.p7s
Description: S/MIME cryptographic signature
Can we use a plugin hook to modify the csp in the bb10 start html page?
localhost is not defined on most platforms afaik, so it may be less of an
issue.
@purplecabbage
risingj.com
On Wed, May 13, 2015 at 1:00 PM, Josh Soref jso...@blackberry.com wrote:
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-windows/pull/76#issuecomment-101816033
LGTM for merge! @purplecabbage for review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
It is and it is failing currently. In the file-transfer plugin [1]
engine name=cordova-wp8 version==4.0.0/
...
Failed to install 'cordova-plugin-file-transfer':CordovaError: Plugin
doesn't support this project's cordova-wp8 version. cordova-wp8: 4.0.0-dev,
failed version requirement: 1.0.0
...
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-js/pull/111
---
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
https://issues.apache.org/jira/browse/CB-9026
-Original Message-
From: Shazron [mailto:shaz...@gmail.com]
Sent: Wednesday, May 13, 2015 4:19 PM
To: dev@cordova.apache.org
Subject: Re: plugins.cordova.io
Good idea. Maybe also file an issue under the 'cordova-registry-web'
component
switching engine to =4.0 (no patch version) seems to make the most sense
for this problem.
On Wed, May 13, 2015 at 4:21 PM, Jesse purplecabb...@gmail.com wrote:
Yes, I just verified the same.
Changing the plugin.xml engine to :
cordova-wp8 =4.0.0-- will allow install for both wp8@4.0.0 and
So back to the initial discussion, should we keep our -dev suffixes?
If so we need to be sure to use engine tags like '=4.0' or if you
specifically want to force a specific min patch, and still allow the dev
suffix, you can do '=4.0.0--'
And of course this should be documented.
@purplecabbage
No, you were correct - any sort of pre-release identifier makes the version
less than the version without the pre-release identifier (since it signifies a
pre-release).
Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and
'4.0.0-iwishiwasarealreease' are all 4.0.0.
It seems to me the problem
Yes, I just verified the same.
Changing the plugin.xml engine to :
cordova-wp8 =4.0.0-- will allow install for both wp8@4.0.0 and
wp8@4.0.0-dev
@purplecabbage
risingj.com
On Wed, May 13, 2015 at 4:18 PM, Tim Barham tim.bar...@microsoft.com
wrote:
No, you were correct - any sort of pre-release
+1 for dropping the suffix - if there is no real good reason for it. I will
make the change to WP8 as it's blocking WP8 CI testing.
Thanks,
Nikhil
-Original Message-
From: Jesse [mailto:purplecabb...@gmail.com]
Sent: Tuesday, May 12, 2015 5:15 PM
To: dev@cordova.apache.org
Subject:
Github user robpaveza commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30254781
--- Diff: src/windows/CameraProxy.js ---
@@ -337,28 +336,48 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args) {
Recently, a issue (CB-9021) popped up where mobile spec was failing because
the semver check failed on a platform due to `-dev` being in the version
and not valid.
I think we should remove -dev from our versions. Right now, after we do a
release, we add -dev to show that the repo is in
Semver should be able to understand '4.0.0-dev' as the valid version string.
There could be some other requirement problems, like =, .
For example, 4.0.0-dev is 4.0.0.
So, if the requirement is =4.0.0, then the version string '4.0.0-dev' will
fail to satisfy the requirement.
Byoungro So
SSG /
I will create a new thread for dropping the suffix
On Wed, May 13, 2015 at 9:20 AM, Nikhil Khandelwal nikhi...@microsoft.com
wrote:
+1 for dropping the suffix - if there is no real good reason for it. I
will make the change to WP8 as it's blocking WP8 CI testing.
Thanks,
Nikhil
Sorry, I got the other way around. :)
4.0.0-dev is 4.0.0.
Byoungro So
SSG / DPD / Mobile Computing and Compilers
Intel Corporation
-Original Message-
From: So, Byoungro
Sent: Wednesday, May 13, 2015 10:37 AM
To: dev@cordova.apache.org
Subject: RE: [DISCUSS] Dropping the -dev suffix
Github user jacobg commented on the pull request:
https://github.com/apache/cordova-plugin-splashscreen/pull/21#issuecomment-101761529
Hi @shazron: I'm still getting the same crash from users after applied the
latest fix. Any ideas? Thanks.
---
If your project is set up for it, you
If any of you downloaded 1.6.0, sorry -- please download the latest 1.6.1
https://github.com/phonegap/ios-deploy/issues/129
On Tue, May 12, 2015 at 3:39 PM, Shazron shaz...@gmail.com wrote:
Release notes: https://github.com/phonegap/ios-deploy/releases/tag/1.6.0
Here we go!
https://github.com/MobileChromeApps/cordova-plugin-background-app/tree/master/example-app
Also makes use of CLI's ability to store plugins platforms in config.xml
:)
On Tue, May 5, 2015 at 2:50 PM, Andrew Grieve agri...@chromium.org wrote:
Hmm, just have mobile chrome apps'
Github user robpaveza commented on the pull request:
https://github.com/apache/cordova-plugin-contacts/pull/62#issuecomment-101757332
Ping Can I get a merge?
---
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 user purplecabbage commented on the pull request:
https://github.com/apache/cordova-plugin-contacts/pull/62#issuecomment-101766786
LGTM.
I'm sure Dooney Evans would approve.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user deslee closed the pull request at:
https://github.com/apache/cordova-plugin-device/pull/33
---
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
GitHub user 007slm opened a pull request:
https://github.com/apache/cordova-lib/pull/216
The concat() method returns a new array.
the ios plist file can not add more than one dict CFBundleURLTypes arrays
when use cordova plugman add plugin in ios platform.
You can merge this pull
Github user shazron commented on the pull request:
https://github.com/apache/cordova-lib/pull/216#issuecomment-101544132
Can you briefly describe what bug this bug fix solves?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30210499
--- Diff: src/windows/CameraProxy.js ---
@@ -438,6 +457,61 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args) {
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30210493
--- Diff: src/windows/CameraProxy.js ---
@@ -438,6 +457,61 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args) {
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30210477
--- Diff: src/windows/CameraProxy.js ---
@@ -438,6 +457,61 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args) {
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-camera/pull/94#discussion_r30210599
--- Diff: src/windows/CameraProxy.js ---
@@ -337,28 +336,48 @@ function takePictureFromCameraWP(successCallback,
errorCallback, args)
Hi Rakesh,
Good to have you on board. You won't have direct repo access --
instead send a pull request against the repos:
https://github.com/apache?utf8=✓query=cordova-
On Wed, May 13, 2015 at 7:01 AM, Rakesh Kalyankar k.rakesh...@gmail.com wrote:
Hello Community
I am Rakesh. I am currently
48 matches
Mail list logo