Re: A Wavy Future

2016-03-20 Thread pablojan
I will be glad to collaborate :) El 20/3/2016 3:36 a. m., Evan Hughes escribió: The cwiki seems to be best place for the time being, anyone wanting to contribute let the mailing list know for writing permissions. https://cwiki.apache.org/confluence/display/WAVE/A+Wavey+Future On Sat, 19 Mar 20

Re: jsinterop 1.0.0-SNAPSHOT missing

2016-09-14 Thread pablojan
I suggest to compare build.gradle with swellrt's one... https://github.com/P2Pvalue/swellrt/blob/master/wave/build.gradle Sorry, I can't do by myself, out for holidays, no laptop. Thanks Enviado con Boxer El 13 sept. 2016 14:31, Ian Dunlop escribió: Hello, I changed the wave/build.gradle to

[GitHub] incubator-wave pull request #17: Update xerces, gwt and guava dependencies

2016-10-23 Thread pablojan
GitHub user pablojan opened a pull request: https://github.com/apache/incubator-wave/pull/17 Update xerces, gwt and guava dependencies This PR fixes gwt building issues. It also update base gwt and guava deps. You can merge this pull request into a Git repository by running

[GitHub] incubator-wave issue #17: Update xerces, gwt and guava dependencies

2016-10-25 Thread pablojan
Github user pablojan commented on the issue: https://github.com/apache/incubator-wave/pull/17 Not sure If you was expecting me to do the merge, I don't have permissions ;) 2016-10-23 19:10 GMT+02:00 Yuri Z. : > *@vega113* approved this pull

Re: [GitHub] incubator-wave issue #17: Update xerces, gwt and guava dependencies

2016-11-01 Thread pablojan
Also GWT 2.8 has been released xD googlewebtoolkit.blogspot.com.es/2016/10/gwt-28-released.html?m=1 El 1 nov. 2016 4:55 a. m., wisebaldone escribió: Github user wisebaldone commented on the issue: https://github.com/apache/incubator-wave/pull/17 Haha, seems inbetween this passing th

Re: Swellrt dependencies and wave dependencies

2016-11-18 Thread pablojan
Hi Evan, Sure, I guess you can tag me, and then I will create issue in swellrt ! El 18 nov. 2016 6:17, Evan Hughes escribió: Also pablo, if i find issues with anything Im changing with wave would you like me to tag you here or to also create an issue on your github/gitter On Fri, 18 Nov 2016 a

Re: [GitHub] incubator-wave issue #17: Update xerces, gwt and guava dependencies

2016-11-24 Thread pablojan
Thanks guys! ;) El 24 nov. 2016 1:22 p. m., Evan Hughes escribió: increasing the committer member count wouldn't be a bad idea :P On Thu, 24 Nov 2016 at 22:06 Yuri Z wrote: > By the way, maybe we should add pablo as committer? > > On Tue, Nov 1, 2016 at 1:02 PM wrote: > > > Also GWT 2.8 has

Re: SuperDev

2016-12-05 Thread pablojan
Hi Darin, super dev mode works properly in the swellrt branch. I should create a patch but in the meantime you can compare wave/build.gradle manually El 5 dic. 2016 7:20 p. m., Darin Willits escribió: Hi Evan, I appreciate any help you can give. I’d also be able/willing to fix some compile e

Re: Podling Report Reminder - December 2016

2016-12-06 Thread pablojan
Thanks Adam, lets include your text in the report. One thing, regarding the editor the issue caused by chrome 54 has been fixed in swellrt... I will send a PR ;) El 5 dic. 2016 16:48, Adam John escribió: Excellent work, Pablo & folks. I took what both of you wrote and combined / formatted..

[GitHub] incubator-wave pull request #18: Fixes taken from SwellRT

2016-12-06 Thread pablojan
GitHub user pablojan opened a pull request: https://github.com/apache/incubator-wave/pull/18 Fixes taken from SwellRT This PR contains some bug fixes taken from SwellRT fork. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-wave issue #19: Removal / Update of Dependencies and other QOL

2017-03-19 Thread pablojan
Github user pablojan commented on the issue: https://github.com/apache/incubator-wave/pull/19 My apologies for the delayed review. I tried the PR and LGTM: Following tasks ran successfully: testAll, jar, run, createDist I performed following tests

[GitHub] incubator-wave pull request #21: Remove atmosphere as websocket's provider

2017-04-17 Thread pablojan
GitHub user pablojan opened a pull request: https://github.com/apache/incubator-wave/pull/21 Remove atmosphere as websocket's provider Atmosphere has been use for last two years to provide a reliable transport layer for the client-server communications. However, at this mome

[GitHub] incubator-wave issue #21: Remove atmosphere as websocket's provider

2017-04-20 Thread pablojan
Github user pablojan commented on the issue: https://github.com/apache/incubator-wave/pull/21 Closing due to changes were made on top of another PR instead of master. Check new PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-wave pull request #21: Remove atmosphere as websocket's provider

2017-04-20 Thread pablojan
Github user pablojan closed the pull request at: https://github.com/apache/incubator-wave/pull/21 --- 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] incubator-wave pull request #23: Removing atmosphere, revert to plain websoc...

2017-04-20 Thread pablojan
GitHub user pablojan opened a pull request: https://github.com/apache/incubator-wave/pull/23 Removing atmosphere, revert to plain websockets Atmosphere has been use for last two years to provide a reliable transport layer for the client-server communications. However, at this

[GitHub] incubator-wave pull request #24: Workaround WAVE-446

2017-05-04 Thread pablojan
Github user pablojan commented on a diff in the pull request: https://github.com/apache/incubator-wave/pull/24#discussion_r114749307 --- Diff: wave/src/main/java/org/waveprotocol/box/server/frontend/WaveViewSubscription.java --- @@ -154,7 +155,15 @@ public synchronized void

[GitHub] incubator-wave pull request #24: Workaround WAVE-446

2017-05-04 Thread pablojan
Github user pablojan commented on a diff in the pull request: https://github.com/apache/incubator-wave/pull/24#discussion_r114749240 --- Diff: wave/src/main/java/org/waveprotocol/box/server/frontend/WaveViewSubscription.java --- @@ -154,7 +155,15 @@ public synchronized void

[GitHub] incubator-wave pull request #24: Workaround WAVE-446

2017-05-04 Thread pablojan
Github user pablojan commented on a diff in the pull request: https://github.com/apache/incubator-wave/pull/24#discussion_r114749381 --- Diff: wave/src/main/java/org/waveprotocol/box/server/frontend/WaveViewSubscription.java --- @@ -154,7 +155,15 @@ public synchronized void

Re: [NOTICE] Wave is Retired - What's next?

2018-02-02 Thread pablojan
I've just joined the google group. Yuri, do you want me to admin the group too? El 1 feb. 2018 5:49 p. m., Yuri Z escribió: I still have admin access to http://www.waveprotocol.org/, however, it looks like the old discussion group was deleted by Google. I recreated it from scratch. So, we can