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

Added Chat support & also displaying the time remaining for each user #9

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

Conversation

kiran9k
Copy link
Contributor

@kiran9k kiran9k commented Jan 21, 2015

Hi ,

In the previous request , the time played by each user was being displayed .But however , I felt that it is important to display the time remaining than the time taken . This pull request is for that purpose . (Also , an option is provided , thru which one can add 10 sec to each valid move by a player .)

Also , a small chat session has been integrated . Whenever anyone messages in the chat , all players are notified & the chat box opens itself . Its just a basic chat done using Socket io . Feel free to change the design or any other components .

Regards,
Kiran

@fmbenhassine
Copy link
Owner

Great! Another cool addition, many thanks Kiran 👍

I'll merge it asap.

For next time, try to send a pull request per feature. This would be easier to follow changes and to keep project's history clean.

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