Skip to content

lab-remil #8

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

remilonwheels
Copy link

Description
[x] Create a TCP Server using the NodeJS native net module
[x] Create a Client Constructor
[x] When sockets connect to the server, a new Client instance should be made
[x] All clients should have a unique id property - this should come from the use of node-uuid
[x] When sockets are connected with the client pool they should be given event listeners for data, error, and close events
[x] When a socket emits the close event, the socket should be removed from the client pool
[x] When a socket emits the error event, the error should be logged on the server
[x] When a socket emits the data event, the data should be logged on the server and the commands below should be implemented

Custom commands

[x] @ALL should trigger a broadcast event
[x] @Nickname should allow a user change their nickname
[x] @dm should allow a user to send a message directly to another user by nick name or by their guest id (unique client id)
[x] when a user sends a message, their nickname should be printed

@@ -0,0 +1,44 @@
'use strict';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done separating out the events into a separate file!

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.

2 participants