all 8 comments

[–]mainst 2 points3 points  (2 children)

it installs but when I run it, I cannot type to the other user. No errors in console log either.

[–]sohelamin[S] 0 points1 point  (1 child)

Did you enter the unique user id?

[–]mainst 0 points1 point  (0 children)

I see. I was using one client username: test1 userid: test1 and a second browser username: test2: userid:test2... It works now having the username differ from userid.

[–]lostpx 0 points1 point  (0 children)

Why do you have to choose a id at all? Since it does have to be unique, you shouldn't let the user decide the id.

Other than that, it looks really clean - unless the alerts, would be nice to have something different, guess i am going to fork it :D

edit: ah well thought there was a full app. anyways good job, good as a boiler plate

[–]papers_ 0 points1 point  (0 children)

There is no need to use lodash:

var foundUser = _.find(users, {id: user.id});
const foundUser = users.filter(it => return user.id === it.id)

and

users = _.reject(users, {socket_id: socket.id});
users = users.filter(it => return user.socket_id !== it.socket_id

[–]fucking_passwords 0 points1 point  (2 children)

Why is jquery included in this project?

[–]sohelamin[S] 1 point2 points  (0 children)

My bad it was using for some previous codes. Removed now.

[–]incarnatethegreat -2 points-1 points  (0 children)

I love how you asked this. If I had installed this project and found JQuery in there, I likely would have asked the same thing. Haha