Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Socket message structure #27

Draft
wants to merge 39 commits into
base: master
Choose a base branch
from

Conversation

foolishsailor
Copy link
Contributor

@foolishsailor foolishsailor commented Apr 23, 2023

*** DRAFT ***

This is a new propsed socket message structure

  • Abstracts logic out of socket object
  • Uses reducers and actions to process all message maintaining type safety
  • Breaks message into 3 categories: Command, State and Message
  • Only the commands are detailed here but a simple copy/paste to flesh out sekeltons for remainder

Have a look as this is the direction Im taking ai-lens and want to make as easy as possible to drop in a socket server module into a devs project and they can just pop in the business logic into the various exposed functions

note: this is built on top of the previous PR. For the purposes of this PR the only relevant changes are within the socket-server folder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant