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

Show the actual number of users online #111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

therohitramesh
Copy link
Collaborator

@therohitramesh therohitramesh commented Feb 24, 2018

Fixes #107

@therohitramesh
Copy link
Collaborator Author

@lunaticmonk
Copy link
Member

@therohitramesh I guess you misunderstood what is required over here. You have to show the users real time on the index page. Currently, we are returning the number of online users to the template. Use sockets instead of that.

@mrmir
Copy link
Contributor

mrmir commented Feb 24, 2018

@therohitramesh You've used the format for opening an issue instead of submitting a PR.

@therohitramesh
Copy link
Collaborator Author

@mrmir Oh yeah. Thanks.

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.

3 participants