LiveSplitOne icon indicating copy to clipboard operation
LiveSplitOne copied to clipboard

Align web socket commands with LiveView server

Open etopiei opened this issue 5 years ago • 4 comments

Currently the web socket commands are not named the same way as those used by the LiveSplit server component. I have written a script to connect the Android Remote app to LiveSplitOne (here) but as some of the commands are different and/or not supplied it is tricky to get this working. In order for more tools to be able to interface better with LiveSplitOne I think it would be good to have standardized commands that are the same as the desktop app.

I am fairly new to this code base, so let me know if anything needs to change or could be done differently, I'm open to feedback and different ideas of how this should be done. This is currently more of a POC, but I'd be happy to help bring this to LiveSplitOne.

etopiei avatar Jul 27 '20 06:07 etopiei

Considering there's all sorts of new functions dedicated entirely to the WebSocket communication, I'd probably also would want to see all of this logic moved out into a new source file.

CryZe avatar Jul 27 '20 06:07 CryZe

@CryZe I've re-worked this a bit as you suggested, hopefully this is looking a bit better now. Let me know if you'd like any other changes or would like to try things another way. Cheers :+1:

etopiei avatar Jul 29 '20 11:07 etopiei

While you've moved some functionality over into WebSocket.ts, I'd actually like to see all the WebSocket logic be moved over. There should be close to nothing left in the TimerView.

CryZe avatar Jul 29 '20 16:07 CryZe

@CryZe Sorry this has taken so long, I've been a bit busy lately, but I've created a new WebSocket manager class which handles the web socket setup and communication. I think this is looking a bit cleaner now, but let me know if there are any further changes required. :+1:

etopiei avatar Aug 11 '20 01:08 etopiei