Github user jcesarmobile commented on a diff in the pull request:
https://github.com/apache/cordova-android/pull/316#discussion_r72573788
--- Diff: framework/src/org/apache/cordova/CordovaActivity.java ---
@@ -175,9 +175,13 @@ protected void createViews() {
setContentVi
Github user jcesarmobile commented on a diff in the pull request:
https://github.com/apache/cordova-android/pull/316#discussion_r72573856
--- Diff: framework/src/org/apache/cordova/CordovaActivity.java ---
@@ -175,9 +175,13 @@ protected void createViews() {
setContentVi
Github user akilawickey commented on a diff in the pull request:
https://github.com/apache/cordova-android/pull/316#discussion_r72578371
--- Diff: framework/src/org/apache/cordova/CordovaActivity.java ---
@@ -175,9 +175,13 @@ protected void createViews() {
setContentVie
Github user akilawickey commented on a diff in the pull request:
https://github.com/apache/cordova-android/pull/316#discussion_r72578456
--- Diff: framework/src/org/apache/cordova/CordovaActivity.java ---
@@ -175,9 +175,13 @@ protected void createViews() {
setContentVie
Github user akilawickey commented on a diff in the pull request:
https://github.com/apache/cordova-android/pull/316#discussion_r72580800
--- Diff: framework/src/org/apache/cordova/CordovaActivity.java ---
@@ -175,9 +175,13 @@ protected void createViews() {
setContentVie
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/623
---
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 i
GitHub user vladimir-kotikov opened a pull request:
https://github.com/apache/cordova-lib/pull/470
CB-11636 Handle attributes with quotes correctly
This PR changes `mergeXml` inner implementation to avoid using
`elementtree.findall()` with complex queries such as
`tag[@attrib1="val
Blog post:
http://cordova.apache.org/announcements/2016/07/27/cordova-windows-4.4.2.html
Tweet: https://twitter.com/apachecordova/status/758621979238887424
-
Vladimir
Github user codecov-io commented on the issue:
https://github.com/apache/cordova-lib/pull/470
## [Current
coverage](https://codecov.io/gh/apache/cordova-lib/pull/470?src=pr) is 80.62%
(diff: 100%)
> Merging [#470](https://codecov.io/gh/apache/cordova-lib/pull/470?src=pr)
into [ma
Github user vladimir-kotikov commented on the issue:
https://github.com/apache/cordova-lib/pull/470
@stevengill, @purplecabbage, could you please 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 your project doe
Github user vladimir-kotikov commented on the issue:
https://github.com/apache/cordova-plugin-camera/pull/223
@sgrebnov, can you take a look?
---
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 t
I vote +1:
* Reviewed release notes
* Verified signatures and hashes
* Verified that blank app could be created with the platform from RC archive
* Verified that blank app could be successfully built and ran
-
Vladimir
-Original Message-
From: Steven Gill [mailto:stevengil...@gmail.com]
I vote +1:
* Reviewed release notes
* Verified signatures and hashes
* Verified that blank app could be created with the platform from RC archive
* Verified that blank app could be successfully built and ran
-
Vladimir
-Original Message-
From: Steven Gill [mailto:stevengil...@gmail.com]
Github user sgrebnov commented on the issue:
https://github.com/apache/cordova-plugin-camera/pull/223
Reviewed, LGTM :+1:
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-camera/pull/223
---
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 cordova-qa commented on the issue:
https://github.com/apache/cordova-plugin-camera/pull/219
Cordova CI Build has one or more failures.
**Commit** -
[Link](https://github.com/apache/cordova-plugin-camera/pull/219/commits/fed798e6c7f56e23bff994ab2c11d316c8257a78
Github user stevengill commented on a diff in the pull request:
https://github.com/apache/cordova-ios/pull/234#discussion_r72683673
--- Diff: bin/templates/scripts/cordova/lib/plugman/pluginHandlers.js ---
@@ -91,18 +94,31 @@ var handlers = {
}
},
GitHub user don opened a pull request:
https://github.com/apache/cordova-plugin-dialogs/pull/80
CB-8947: (ios)
### Platforms affected
iOS
### What does this PR do?
Fix crash when non-strings are passed to dialogs
### What testing has been done on
Github user jcesarmobile commented on the issue:
https://github.com/apache/cordova-plugin-media-capture/pull/65
Why do you use medium quality for value 0? shouldn't you use Low?
I think you should use:
0 - low
1 - high
no quality passed - medium (default)
---
If
Github user dpogue commented on a diff in the pull request:
https://github.com/apache/cordova-lib/pull/469#discussion_r72685014
--- Diff: cordova-lib/src/cordova/metadata/android_parser.js ---
@@ -31,14 +31,20 @@ var fs= require('fs'),
function android_p
Github user cordova-qa commented on the issue:
https://github.com/apache/cordova-plugin-dialogs/pull/80
Cordova CI Build has completed successfully.
**Commit** -
[Link](https://github.com/apache/cordova-plugin-dialogs/pull/80/commits/fb8e6bc8a8bff0bfd3c783bce6be286192e1a95
Github user infil00p commented on a diff in the pull request:
https://github.com/apache/cordova-lib/pull/469#discussion_r72688890
--- Diff: cordova-lib/src/cordova/metadata/android_parser.js ---
@@ -31,14 +31,20 @@ var fs= require('fs'),
function android
GitHub user dpogue opened a pull request:
https://github.com/apache/cordova-android/pull/321
CB-8978 Prepare copy resource-files from config.xml
See also cordova/cordova-discuss#6
Depends on https://github.com/apache/cordova-lib/pull/468
This copies files specified by ``
Github user codecov-io commented on the issue:
https://github.com/apache/cordova-android/pull/321
## [Current
coverage](https://codecov.io/gh/apache/cordova-android/pull/321?src=pr) is
33.85% (diff: 100%)
> Merging
[#321](https://codecov.io/gh/apache/cordova-android/pull/321?src=
Github user philbenoit commented on the issue:
https://github.com/apache/cordova-plugin-contacts/pull/130
Any word on when this might be merged or if I need to provide any more
details?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user jcesarmobile commented on the issue:
https://github.com/apache/cordova-plugin-dialogs/pull/80
Hi.
I think you should edit the README.md to document the changes commenting
that accept any type but will be stringified or something like that.
And maybe add this to
Github user don commented on the issue:
https://github.com/apache/cordova-plugin-dialogs/pull/80
@jcesarmobile passing strings as the message is the *right* thing to do,
IMO. My main goal was to stop iOS from crashing when the user sent bad data. I
think the documentation should still
GitHub user juliascript opened a pull request:
https://github.com/apache/cordova-coho/pull/129
CB-11642 Update CDVAvailability.h
See https://issues.apache.org/jira/browse/CB-11642
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/j
Nightly build #85 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/85/consoleFull
-
Jenkins for Apache Cordova
---
Github user ryanwilliams83 commented on the issue:
https://github.com/apache/cordova-plugin-media-capture/pull/65
UIImagePickerControllerQualityTypeMedium is the default value of the
UIImagePickerController.videoQuality property and what is used in the current
version of this plugin.
30 matches
Mail list logo
