Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Breaking change: new Socket API system #60

Merged
merged 1 commit into from
Jan 23, 2017
Merged

Breaking change: new Socket API system #60

merged 1 commit into from
Jan 23, 2017

Conversation

KalleV
Copy link
Contributor

@KalleV KalleV commented Jan 23, 2017

Update the SocketIOLoader to load socket connection handlers instead
of simply passing the connected socket to an 'onConnect' function. The
connection handler objects allow the developer of a socket API to specify
the following fields:

  • event: the event name
  • onEvent: the main event handler
  • middleware: an optional array of connect-style middleware that run before the onEvent handler is called

Additional:

  • Update documentation for the new socket API creation system.
  • Update tests

Update the SocketIOLoader to load socket connection handlers instead
of simply passing the connected socket to an 'onConnect' function. The
connection handler objects allow the developer of a socket API to specify
the following fields:
 - event: the event name
 - onEvent: the main event handler
 - middleware: an optional array of connect-style middleware that run before the onEvent handler is called

Additional:
 - Update documentation for the new socket API creation system.
 - Update tests
@KalleV KalleV merged commit d4d625b into master Jan 23, 2017
@KalleV KalleV deleted the new-socket-api branch January 23, 2017 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants