Thanks,

On 18 March 2014 19:33, Vicente J. Ruiz Jurado <v...@ourproject.org> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18941/
>
> On March 9th, 2014, 8:34 p.m. UTC, *Ali Lown* wrote:
>
> LGTM.
> Can I request that you document this new mode in 
> https://cwiki.apache.org/confluence/display/WAVE/Client+Development+Setup 
> once you commit this patch?
>
>  On March 13th, 2014, 11:25 p.m. UTC, *Vicente J. Ruiz Jurado* wrote:
>
> Off course (Can you grant me permissions?).
>
> Commited: 3682cf9146cfc975eeabc77757aeccde5389edcb
>
>  Documentation updated: 
> https://cwiki.apache.org/confluence/display/WAVE/Client+Development+Setup
>
>
> - Vicente J.
>
> On March 8th, 2014, 12:42 p.m. UTC, Vicente J. Ruiz Jurado wrote:
>   Review request for wave, Ali Lown, Pablo Ojanguren, and Yuri Zelikov.
> By Vicente J. Ruiz Jurado.
>
> *Updated March 8, 2014, 12:42 p.m.*
>  *Bugs: * WAVE-384 <https://issues.apache.org/jira/browse/WAVE-384>
>  *Repository: * wave
> Description
>
> This patch add the GWT superdev mode to 'waveharness' and 'editor' that now 
> works again. A refactorization is needed because some new functionalities had 
> broken these GWT dev tools (WAVE-384).
>
> Note: GWT superdev mode in the full client (ant superdev-gwt) does not work 
> yet because we need to replace socketio-java GWT dependency with an 
> alternative (like atmosphere) or patched it with
>
> https://github.com/vjrj/Socket.IO-Java/commit/f7010aee1f5cc5adea0aab3f27e48c37d1b10b5e
> I think is more interesting the first option and seems that pablojan is 
> working on it, because socketio-java update to jetty 9 websocket (#WAVE-405) 
> version is not well 
> documented:https://github.com/jetty-project/jetty-documentation/issues/12
> Anyway, when the replacement is ready, we can just uncomment the xsiframe 
> linker in WebClientDev.gwt.xml and Editor.gwt.xml.
>
>   Testing
>
> Run:
> ant waveharness-superdev
> Add the bookmarks in: http://localhost:9876/
> and play withhttp://localhost:9876/waveharness/UndercurrentHarness.html
> so some code changes (for instance in UndercurrentHarness.java), click in 
> "Dev mode on" in the bookmarks and recompile the module.
>
> Same with:
> ant editor-superdev
>
>   Diffs
>
>    - build.xml (0a1b23c)
>    - src/org/waveprotocol/box/webclient/WebClientDev.gwt.xml (da32dd7)
>    - src/org/waveprotocol/box/webclient/client/ClientEvents.java (8b8e411)
>    - src/org/waveprotocol/box/webclient/client/DebugMessagePanel.java
>    (def986e)
>    - src/org/waveprotocol/box/webclient/client/HistoryChangeListener.java
>    (ddcdb5d)
>    - src/org/waveprotocol/box/webclient/client/HistoryProvider.java
>    (f207b83)
>    - src/org/waveprotocol/box/webclient/client/RemoteWaveViewService.java
>    (8d4ff15)
>    - src/org/waveprotocol/box/webclient/client/SavedStateIndicator.java
>    (3ee5c74)
>    - src/org/waveprotocol/box/webclient/client/SimpleWaveStore.java
>    (83484c4)
>    - src/org/waveprotocol/box/webclient/client/StagesProvider.java
>    (dfca652)
>    - src/org/waveprotocol/box/webclient/client/WaveWebSocketClient.java
>    (3bb39a2)
>    - src/org/waveprotocol/box/webclient/client/WebClient.java (d3e3b49)
>    - src/org/waveprotocol/box/webclient/client/WindowTitleHandler.java
>    (ff1654f)
>    - src/org/waveprotocol/box/webclient/client/events/DebugMessageEvent.java
>    (6245489)
>    - 
> src/org/waveprotocol/box/webclient/client/events/DebugMessageEventHandler.java
>    (57f4659)
>    - src/org/waveprotocol/box/webclient/client/events/Log.java (af132fe)
>    - src/org/waveprotocol/box/webclient/client/events/NetworkStatusEvent.java
>    (e495cc0)
>    - 
> src/org/waveprotocol/box/webclient/client/events/NetworkStatusEventHandler.java
>    (034d062)
>    - src/org/waveprotocol/box/webclient/client/events/WaveCreationEvent.java
>    (ec59ccc)
>    - 
> src/org/waveprotocol/box/webclient/client/events/WaveCreationEventHandler.java
>    (3ddf1a0)
>    - src/org/waveprotocol/box/webclient/client/events/WaveOpenEvent.java
>    (b5dcd75)
>    - 
> src/org/waveprotocol/box/webclient/client/events/WaveOpenEventHandler.java
>    (f440d29)
>    - src/org/waveprotocol/box/webclient/client/events/WaveSelectionEvent.java
>    (a86469d)
>    - 
> src/org/waveprotocol/box/webclient/client/events/WaveSelectionEventHandler.java
>    (a35242c)
>    - src/org/waveprotocol/box/webclient/search/SimpleSearch.java (c79af80)
>    - src/org/waveprotocol/box/webclient/search/WaveBasedDigest.java
>    (8c04db8)
>    - src/org/waveprotocol/box/webclient/search/WaveContext.java (c8deb2a)
>    - src/org/waveprotocol/box/webclient/search/WaveStore.java (2290737)
>    - src/org/waveprotocol/wave/client/StageTwo.java (557e838)
>    - 
> src/org/waveprotocol/wave/client/common/util/UserAgentStaticProperties.java
>    (0bca984)
>    - 
> src/org/waveprotocol/wave/client/doodad/attachment/AttachmentManagerProvider.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/editor/Editor.gwt.xml (78f3c26)
>    - src/org/waveprotocol/wave/client/events/ClientEvents.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/DebugMessageEvent.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/DebugMessageEventHandler.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/Log.java (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/NetworkStatusEvent.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/NetworkStatusEventHandler.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/WaveCreationEvent.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/WaveCreationEventHandler.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/WaveOpenEvent.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/WaveOpenEventHandler.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/WaveSelectionEvent.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/events/WaveSelectionEventHandler.java
>    (PRE-CREATION)
>    - src/org/waveprotocol/wave/client/testing/UndercurrentHarness.gwt.xml
>    (e48fd59)
>    - src/org/waveprotocol/wave/client/testing/UndercurrentHarness.java
>    (22f7df6)
>    - 
> src/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantController.java
>    (21995a6)
>    - src/org/waveprotocol/wave/model/conversation/TitleHelper.java
>    (9402d67)
>    - src/org/waveprotocol/wave/model/document/WaveContext.java
>    (PRE-CREATION)
>
> View Diff <https://reviews.apache.org/r/18941/diff/>
>

Reply via email to