Created an issue: https://issues.apache.org/jira/browse/INFRA-10718
On Wed, Nov 4, 2015 at 5:06 AM, Carlos Santana wrote:
> Did someone opened a infra ticket?
>
> Using the git-wip url on platform add should work right?
> I usually use content of dist when verifying and
DefinitelyTyped(https://github.com/borisyankov/DefinitelyTyped) is a good
example IMO of a repo that manages to deal with an absurd ammount of issues
and PRs in a very good way, although I agree that it doesn't "suffer" from
the fact that cordova is a sum of a lot of independent modules/projects,
Problem with collecting from lib is how do you show notice to users to opt
out? If we build that notice into lib, imagine how confusing that might be
for PhoneGap/Taco/Mobilefirst users to see a message about cordova wanting
to collect data.
I think we should only introduce it to CLI if we do.
+1 and do minor release since it include more than bug fixes.
On Tue, Nov 3, 2015 at 12:10 PM Jesse wrote:
> +1
> Sounds good. I want to test some of the recent line-ending gitattribute
> changes with AppStore submission, so a release would be the perfect test.
>
> > On
user space +1 plugin land
On Tue, Nov 3, 2015 at 4:39 PM Parashuram N wrote:
> +1 to making this a plugin. I think folks in the community may be
> interested in helping creating the plugin. Is that what you are looking for
> ?
>
> -Original Message-
> From:
LGTM left a question.
On Tue, Nov 3, 2015 at 6:36 PM Shazron wrote:
> Sent a PR https://github.com/apache/cordova-ios/pull/178 please
> review/comment etc
>
> On Thu, Oct 29, 2015 at 7:27 AM, julio cesar sanchez
> wrote:
> > +1, I first saw the
currently evaluating with some other folks at work, will provide feedback
soon.
On Tue, Nov 3, 2015 at 11:07 PM Tommy-Carlos Williams
wrote:
> +1 to letting the OS handle it.
>
> > On 4 Nov 2015, at 12:44, Jesse wrote:
> >
> > I completely support
Thanks for your feedback Julien!
We absolutely DO use github, and most of our conversation happens there, or
here on the dev list ( which also echoes every github code comment, pull
request discussion )
Some thoughts:
- github issues would mean we would have 30+ issue trackers, one for every
This should really go to discuss if this doesn't have the vote in it.
On Wed, Nov 4, 2015 at 12:50 PM Steven Gill wrote:
> Created an issue: https://issues.apache.org/jira/browse/INFRA-10718
>
> On Wed, Nov 4, 2015 at 5:06 AM, Carlos Santana
>
Yep, no collection on app just tools.
And also be careful on what we collect on tooling also, there might things
we might not want to collect. (cli commands might have sensitive data) we
just need to collect some of it in a anonymous way.
Also some environment info also is useful (node version,
Thanks Julien!
I think are definitely things we could do better in terms of integrating
with github.
Some pain points:
- Can't label PRs
- Can't merge commits from the GitHub interface (great for small commits)
- Apache infra mirroring is flaky. Changes to things like tags don't get
reflected
Please review and vote on this Tools Release
by replying to this email (and keep discussion on the DISCUSS thread)
Release issue: https://issues.apache.org/jira/browse/CB-9903
Tools have been published to
dist/dev:https://dist.apache.org/repos/dist/dev/cordova/CB-9903/
The packages were
I didn't mention the obvious that having Github as issues frontend will
bring a lot of noise, and current commiters don't necessarly have time to
handle this, which i perfectly understand.
Maybe the companies fueling Cordova, and those using it could help by
financing someone from our community
This vote has failed.
On Wed, Nov 4, 2015 at 12:03 PM, Steven Gill wrote:
> Ugh, just realized I didn't update app-hello-world in
> https://github.com/apache/cordova-lib/blob/master/cordova-lib/src/platforms/platformsConfig.json#L53
>
> Think I have to restart this vote
Github user csantanapr commented on the pull request:
https://github.com/apache/cordova-ios/pull/178#issuecomment-153887028
With this removal, is there a minimum ios version that this will work only
using the IFRAME_NAV bridge?
---
If your project is set up for it, you can reply to
Sorry Joe, I should know better :-(
On Wed, Nov 4, 2015 at 3:58 PM Joe Bowser wrote:
> This should really go to discuss if this doesn't have the vote in it.
>
> On Wed, Nov 4, 2015 at 12:50 PM Steven Gill
> wrote:
>
> > Created an issue:
Github user ghenry22 commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/143#issuecomment-153923917
CB-9020 does say they are targeting WP 8.1 which uses the new windows
platform so this PR would also resolve CB-9020.
CB8257 seems to refer
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file/pull/141
---
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-plugin-file/pull/143#issuecomment-153929493
Done! Thanks ghenry22 !
---
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 asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file/pull/121
---
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 asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file/pull/143
---
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 asfgit closed the pull request at:
https://github.com/apache/cordova-cli/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 feature
On this same vein: may I do the reduction in Cordova versions, from 3.1.0,
3.4.0, 3.5.0 to just 3.5 (or whatever the latest was)? This will result in some
HTTP 302 redirects.
Kindly,
Dmitry
> On Nov 4, 2015, at 3:19 PM, Carlos Santana wrote:
>
> check has value, and is
Github user ghenry22 commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/141#issuecomment-153925590
I reported this and can confirm it is the case on the windows platform.
This is a trivial change to the docs but very useful information for anyone
I don't know - check to me just sounds nicer. Plus, I didn't know
about outdated. ;)
On Wed, Nov 4, 2015 at 5:03 PM, Carlos Santana wrote:
> Forgot to include npm outdated reference https://docs.npmjs.com/cli/outdated
> very useful I use it a lot
>
> On Wed, Nov 4, 2015 at
check has value, and is different functionality from "outdated"
On Wed, Nov 4, 2015 at 6:05 PM Raymond Camden
wrote:
> I don't know - check to me just sounds nicer. Plus, I didn't know
> about outdated. ;)
>
> On Wed, Nov 4, 2015 at 5:03 PM, Carlos Santana
I meant it's an opt-in, it's off by default in cordova-lib
CLI will tell lib to turn on collection, other CLIs can choose to collect
or not to collect and tell Lib
On Wed, Nov 4, 2015 at 6:01 PM Steven Gill wrote:
> Problem with collecting from lib is how do you show
Github user shazron commented on the pull request:
https://github.com/apache/cordova-ios/pull/178#issuecomment-153908591
IFRAME_NAV has always worked from the beginning and has been the default,
the other bridge modes are experimental to try to gain speed, but flakey --
like the
Please review the blog post and provide feedback on the PR.
https://github.com/apache/cordova-docs/pull/411
On Mon, Nov 2, 2015 at 5:26 PM, Steven Gill wrote:
> I'm redoing the vote thread for tools and including the node 5.0.0 fix.
>
Github user jasongin commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/143#issuecomment-153923115
Sorry I mistyped... I meant to refer to CB-9020 not CB-9090.
I was thinking CB-8257 was referring to WP 8.1 (which uses the 'windows'
platform for
GitHub user riknoll opened a pull request:
https://github.com/apache/cordova-docs/pull/412
CB-9959: Makes plugins search case insensitive
@dblotsky please review
You can merge this pull request into a Git repository by running:
$ git pull
Github user ghenry22 commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/143#issuecomment-153925057
MS docs confirm the same, should be ms-appx:///
Committing an update to fix now.
reference:
Github user stevengill commented on the pull request:
https://github.com/apache/cordova-docs/pull/409#issuecomment-153926378
Joe, going to commit this in after I do a 5.4.0 docs release (since android
4 won't be accessible until next tools release)
---
If your project is set up for
Github user jasongin commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-file/pull/143#discussion_r43965498
--- Diff: src/windows/FileProxy.js ---
@@ -168,6 +168,15 @@ WinFS.prototype.__format__ = function(fullPath) {
return
Forgot to include npm outdated reference https://docs.npmjs.com/cli/outdated
very useful I use it a lot
On Wed, Nov 4, 2015 at 6:02 PM Carlos Santana wrote:
> I would like to see a command `$cordova outdated` similar to the `$npm
> outdated` [1]
>
> I open a JIRA Feature
I would like to see a command `$cordova outdated` similar to the `$npm
outdated` [1]
I open a JIRA Feature here [2] shows the concept I'm looking for
https://issues.apache.org/jira/browse/CB-9955
On Wed, Nov 4, 2015 at 2:04 PM Shazron wrote:
>
GitHub user stevengill opened a pull request:
https://github.com/apache/cordova-docs/pull/411
CB-9903 added 5.4.0 tools release blog
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/stevengill/cordova-docs master
Alternatively
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-js/pull/129
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-lib/pull/307
---
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 ghenry22 commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/143#issuecomment-153922252
CB-9090 is for the whitelist plugin
CB-8257 is the same issue but for the wp8 platform. This fix is for the
windows platform, I don't have native
Github user ghenry22 commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/143#issuecomment-153926046
once merged you could also close out PR #121 as duplicate as it attempts to
solve the same issue but generates absolute paths from c:\... which are not
Fixed deficiencies in the flowcharts, updated:
https://github.com/shazron/cordova-discuss/blob/ios-remove-whitelist/proposals/ios-whitelist-removal.md
On Wed, Nov 4, 2015 at 2:42 PM, Carlos Santana wrote:
> currently evaluating with some other folks at work, will provide
Github user ghenry22 commented on the pull request:
https://github.com/apache/cordova-plugin-media/pull/62#issuecomment-153949837
PR #12 & PR #18 could also be closed out as duplicate once this one is
merged which will help to clean up some older lingering PR's
---
If your project
Github user nikmartin commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/112#issuecomment-153958937
On a Samsung S5 with Android 5.0.1 this patch works fine
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user sgrebnov commented on the pull request:
https://github.com/apache/cordova-docs/pull/411#issuecomment-153961020
lgtm
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user dblotsky opened a pull request:
https://github.com/apache/cordova-docs/pull/413
Switch to /latest
Switching to using /latest URI by default. Fixing 404 page to recognize it.
You can merge this pull request into a Git repository by running:
$ git pull
Github user dblotsky commented on the pull request:
https://github.com/apache/cordova-docs/pull/413#issuecomment-153962594
@riknoll 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 does not
Github user mgtorrea commented on the pull request:
https://github.com/apache/cordova-lib/commit/1b55e26bc5b234aa5c988291eee010607dba7c00#commitcomment-14201499
@rainergp I've had that same problem. I installed node 4.2.2 and the
problem went away.
---
If your project is set up for
Github user sgrebnov commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/411#discussion_r43977459
--- Diff: www/_posts/2015-11-04-tools-release.md ---
@@ -0,0 +1,121 @@
+---
+layout: post
+author:
+name: Steve Gill
+url:
Github user infil00p commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/112#issuecomment-153969952
@charlesverge, do you have an ICLA signed for this? Usually when people
write entire new methods, I need an ICLA before I can accept public
Ugh, just realized I didn't update app-hello-world in
https://github.com/apache/cordova-lib/blob/master/cordova-lib/src/platforms/platformsConfig.json#L53
Think I have to restart this vote again.
On Tue, Nov 3, 2015 at 11:18 AM, Sergey Grebnov (Akvelon) <
v-seg...@microsoft.com> wrote:
> +1
>
>
Dear cordova commiters, community, users,
Please let me raise another red flag here about not fully embracing GitHub
for the Cordova project.
I'm just a long time cordova user, fanboy, local evangelist, meetup
organiser... but i can feel the daily users frustration when facing
official cordova
Fixed now btw
On Wed, Nov 4, 2015 at 10:39 AM, Steven Gill wrote:
> Oh it was updated, but a newer commit deleted my update from last night.
> I'm adding it back in now.
>
> On Wed, Nov 4, 2015 at 10:34 AM, Frederico Galvão <
> frederico.gal...@pontoget.com.br> wrote:
>
https://issues.apache.org/jira/browse/CB-9951
https://issues.apache.org/jira/browse/CB-9952
https://issues.apache.org/jira/browse/CB-9953
On Wed, Nov 4, 2015 at 10:31 AM, Raymond Camden wrote:
> Ok, about to file some bugs, and thanks Steven for your post too. When
> I
Thanks!
2015-11-04 17:14 GMT-02:00 Steven Gill :
> Fixed now btw
>
> On Wed, Nov 4, 2015 at 10:39 AM, Steven Gill
> wrote:
>
> > Oh it was updated, but a newer commit deleted my update from last night.
> > I'm adding it back in now.
> >
> > On
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file-transfer/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
GitHub user daserge opened a pull request:
https://github.com/apache/cordova-plugin-file-transfer/pull/111
CB-9950 Unpend Filetransfer spec.27 on wp8 as custom headers are now â¦
â¦supported
[Jira issue](https://issues.apache.org/jira/browse/CB-9950)
You can merge this
Github user jcesarmobile commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153756910
I think we should discuss this change before merging, it's a big change and
add an ios "quirk". We recently rejected another pull request that was
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153758198
Yep, I just wanted to clarify on the docs-gen-related things.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153722564
@convey-mane, You are contributing a significant change, so an ICLA/CCLA
need to be signed as well: http://www.apache.org/licenses/#clas
Github user ZombieProtectionAgency closed the pull request at:
https://github.com/apache/cordova-lib/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
Did someone opened a infra ticket?
Using the git-wip url on platform add should work right?
I usually use content of dist when verifying and after release just use npm
- Carlos
@csantanapr
> On Nov 3, 2015, at 4:15 PM, Steven Gill wrote:
>
> The wrappy error still
Github user convey-mane commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153718521
OK with that `npm run gen-docs` the README.me now again is identical to
before your first comment :wink: but nevertheless learned something new.
We would collect data from cli use, not in app.
cli and lib versions, plugins used, cli commands used, platforms used and
versions, etc
NOTHING should be added to the app a developer installs on the emulator or
device.
Ideally this is all done in the cli directly so other tools can still
1) Wow! Cool! But it doesn't seem to work for me. I tried it on a
somewhat old project with just iOS 3.6.3:
android @ 4.1.1; current did not install, and thus its version cannot
be determined
ios @ 3.6.3; current did not install, and thus its version cannot be determined
3) Will it match the CLI
1) cordova platform check
(should be in the help)
2) don't think there is anything for plugins
3) The version has not been updated. There was talk at the F2F to
better update it wrt versions and the dropdown
On Wed, Nov 4, 2015 at 8:34 AM, Raymond Camden wrote:
> I'm
I'm working on a blog post that talks about how to determine what
version of a platform/plugin you have and how to update it. I've got a
few questions regarding this process:
1) Today (or last night :) an updated version of the iOS platform was
released. How do I determine, via the CLI, what the
1) And to make things more confusing. I get different results if I do
cordova platforms check
(ie plural)
With that I get:
android @ 4.1.1; current did not install, and thus its version cannot
be determined
browser @ 3.5.2; current did not install, and thus its version cannot
be determined
ios
1) I'm confused then. On another project, I got this:
add to body class: platform-browser
android @ 4.1.1; current did not install, and thus its version cannot
be determined
browser @ 3.5.2 could be updated to: 4.0.0
ios @ 3.9.1; current did not install, and thus its version cannot be determined
Docs will match cli version. Going to do a docs release for the 5.4.0 tools
release.
I just use npm info package
Would be useful to add a cordova check that showed what versions the cli
will pull.
Btw, the plan is to move towards stop advising users to update platforms
independently (except
Here's the [DISCUSS] thread for docs going forward:
http://markmail.org/message/4sl773axlu372hll
On Wed, Nov 4, 2015 at 8:59 AM, Steven Gill wrote:
> Docs will match cli version. Going to do a docs release for the 5.4.0 tools
> release.
>
> I just use npm info package
>
1) I had 3.9.1 ios installed, it gave me this:
$ cordova platform check
iOS project created with cordova-ios@3.9.2
The message could be better though.
3) Probably not, since we don't update the docs with each CLI release.
Probably just say its the "latest".
On Wed, Nov 4, 2015 at 8:46 AM,
Thanks to everyone who helped get this release out!
Blog post:
http://cordova.apache.org/announcements/2015/11/02/cordova-ios-3-9-2.html
Tweet: https://twitter.com/apachecordova/status/661837413753335808
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-file-transfer/pull/107#issuecomment-153656740
@jeroenv, I agree to @purplecabbage' reply to the dev-list:
> Post a picture that fails so we can verify.
I have tested this with
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-file-transfer/pull/110#issuecomment-153639204
@purplecabbage, should not it be handled by
Github user convey-mane commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153643852
So ⦠just to get it right ⦠please forgive any "newbie"-questions:
I went to my local GitHub-directory, there into the camera-repository,
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153645252
@convey-mane, that's right, just run `npm run gen-docs` after that.
I'm going to merge in #133 soon, please rebase on it after the merge;
the PR
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/133#issuecomment-153646784
Merged in f5217bf02b9018f94ea7c3694a1172fd9a5b97ac
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user daserge closed the pull request at:
https://github.com/apache/cordova-plugin-camera/pull/133
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-153646878
Merged #133
---
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
I think the same, this could be very useful. But we have to think about the
things terence already mentioned. These are three very big points for data
collection and they have to be 100% clear before we get started with this. I
think that we don’t need to collect user identifiable information,
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-windows/pull/129
---
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 daserge commented on the pull request:
https://github.com/apache/cordova-lib/commit/1b55e26bc5b234aa5c988291eee010607dba7c00#commitcomment-14175863
I believe it will be shipped in 5.4.0, which is not yet published.
---
If your project is set up for it, you can reply to
Oh it was updated, but a newer commit deleted my update from last night.
I'm adding it back in now.
On Wed, Nov 4, 2015 at 10:34 AM, Frederico Galvão <
frederico.gal...@pontoget.com.br> wrote:
> http://cordova.apache.org/blog/ isn't being updated automatically, if that
> was ever the desired
Ok, about to file some bugs, and thanks Steven for your post too. When
I blog on this, I'm going to quote you so folks (hopefully) know that
things will be different later.
On Wed, Nov 4, 2015 at 11:07 AM, Shazron wrote:
> Here's the [DISCUSS] thread for docs going forward:
>
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file/pull/145
---
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 asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file/pull/144
---
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 asfgit closed the pull request at:
https://github.com/apache/cordova-ios/pull/177
---
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-plugin-file-transfer/pull/110#issuecomment-153662666
Oops. I guess I was working from an older version of the file and didn't
see the merge happen. I'll fix it.
> On Nov 4, 2015, at
posted the link on the google group
2015-11-04 10:30 GMT+01:00 Steven Gill :
> Thanks to everyone who helped get this release out!
>
> Blog post:
> http://cordova.apache.org/announcements/2015/11/02/cordova-ios-3-9-2.html
> Tweet:
90 matches
Mail list logo