Github user shazron commented on the pull request:
https://github.com/apache/cordova-ios/pull/157#issuecomment-137696927
Hi can you add a prefix of "CB-9610" to the title to enable Apache's JIRA
issue tracker auto-magic linking?
https://issues.apache.org/jira/browse/CB-9610
---
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-vibration/pull/38#issuecomment-137694325
@nikhilkh, @robpaveza thanks for review!
Please take a look once more - I have addressed your comments.
---
If your project is set up for it, you
GitHub user masahirotanaka opened a pull request:
https://github.com/apache/cordova-docs/pull/332
Added Monaca and Onsen logo and description.
Hi. I added some descriptions for the new Cordova Website. Thanks!
You can merge this pull request into a Git repository by running:
$
Github user vladimir-kotikov closed the pull request at:
https://github.com/apache/cordova-plugin-media/pull/65
---
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,
Github user purplecabbage commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/119#discussion_r38773249
--- Diff: template/cordova/lib/log.js ---
@@ -30,33 +30,87 @@ var platformRoot = path.join(__dirname, '..', '..'),
configPath =
Hi guys,
Please review a proposal on plugin documentation generation:
https://github.com/cordova/cordova-discuss/issues/15
Best regards,
Sergey Shakhnazarov
GitHub user kurtisf opened a pull request:
https://github.com/apache/cordova-plugin-dialogs/pull/57
On iOS8+ allow this dialog to be presented from alternate web views.
- On iOS8+ allow this dialog to be presented from alternate web views such
as the Cordova InAppBrowser. Finds
Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-plugin-dialogs/pull/57#issuecomment-137843882
No. InAppBrowser should not be able to call cordova native functions.
---
If your project is set up for it, you can reply to this email and have
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-ios/pull/156
---
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 user asfgit closed the pull request at:
https://github.com/apache/cordova-ios/pull/158
---
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 user purplecabbage commented on the pull request:
https://github.com/apache/cordova-ios/pull/154#issuecomment-137843411
I have essentially done the same thing in #158, but for build/log/run/clean
---
If your project is set up for it, you can reply to this email and have your
Github user kurtisf commented on the pull request:
https://github.com/apache/cordova-plugin-dialogs/pull/57#issuecomment-137844576
The use case is the app receives and event, such as a push notification
while the InAppBrowser is presented. If you use this plugin
Github user daserge closed the pull request at:
https://github.com/apache/cordova-ios/pull/154
---
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 user cleiton-tavares opened a pull request:
https://github.com/apache/cordova-plugin-whitelist/pull/10
remove alert msg in console for develop app
You can merge this pull request into a Git repository by running:
$ git pull
Github user omefire commented on the pull request:
https://github.com/apache/cordova-lib/pull/292#issuecomment-137912540
CB-9560 is merged as well.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user omefire commented on the pull request:
https://github.com/apache/cordova-lib/pull/288#issuecomment-137900148
Merged !
---
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 user asfgit closed the pull request at:
https://github.com/apache/cordova-lib/pull/288
---
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 user cleiton-tavares closed the pull request at:
https://github.com/apache/cordova-plugin-whitelist/pull/10
---
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
Github user omefire commented on the pull request:
https://github.com/apache/cordova-lib/pull/292#issuecomment-137877915
@marcuspridham , did you sign the
[ICLA](http://www.apache.org/licenses/#clas) ?
---
If your project is set up for it, you can reply to this email and have your
Confirmed here: https://people.apache.org/committer-index.html
Middle name and all ...
My team is hiring!
@purplecabbage
risingj.com
On Fri, Sep 4, 2015 at 4:36 PM, marcuspridham wrote:
> Github user marcuspridham commented on the pull request:
>
>
Github user marcuspridham commented on the pull request:
https://github.com/apache/cordova-lib/pull/292#issuecomment-137878645
Yep. I signed the ICLA awhile ago. I have done some pull requests to
Cordova before.
---
If your project is set up for it, you can reply to this email and
Github user marcuspridham commented on the pull request:
https://github.com/apache/cordova-lib/pull/292#issuecomment-137896650
Thanks! Please look at merging CB-9560 when you time.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user marcuspridham closed the pull request at:
https://github.com/apache/cordova-lib/pull/292
---
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 omefire commented on the pull request:
https://github.com/apache/cordova-lib/pull/292#issuecomment-137895107
#merged !
---
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
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/119#discussion_r38726887
--- Diff: template/cordova/lib/log.js ---
@@ -30,33 +30,87 @@ var platformRoot = path.join(__dirname, '..', '..'),
configPath =
Github user alsorokin commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/119#discussion_r38726303
--- Diff: template/cordova/lib/log.js ---
@@ -30,33 +30,87 @@ var platformRoot = path.join(__dirname, '..', '..'),
configPath =
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/119#discussion_r38726309
--- Diff: template/cordova/lib/log.js ---
@@ -30,33 +30,87 @@ var platformRoot = path.join(__dirname, '..', '..'),
configPath =
Github user daserge commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-vibration/pull/38#discussion_r38724321
--- Diff: src/windows/VibrationProxy.js ---
@@ -60,12 +61,43 @@ if (VibrationDevice) {
fail(e);
}
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/119#discussion_r38726108
--- Diff: template/cordova/lib/log.js ---
@@ -30,33 +30,87 @@ var platformRoot = path.join(__dirname, '..', '..'),
configPath =
Github user zalun commented on the pull request:
https://github.com/apache/cordova-lib/commit/07271a5c6162c0b2bee55e1ea23f91ebdfbbb34c#commitcomment-13064163
Hey, I had this pull request which wasn't reviewed yet [1] - it was this
line [2]
I'm wondering where to look to fix it
Github user vladimir-kotikov commented on the pull request:
https://github.com/apache/cordova-lib/commit/07271a5c6162c0b2bee55e1ea23f91ebdfbbb34c#commitcomment-13064620
@zalun, the code you searching for is now here:
31 matches
Mail list logo