-
Notifications
You must be signed in to change notification settings - Fork 26
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
after a hot code push, user isnt online till he hits hard refresh #4
Comments
I can't reproduce using https://github.com/Konecty/meteor-user-presence-example-chat |
Maybe it has something to do with me using meteor-cluster and making use of multiple cores. |
Probably, can you confirm? I don't know if this package works with meteor-cluster |
Well it seems to be working fine for me (in general). I have it running on The reason I even made the switch over to this package was the fact that
|
Yes, we've used the mizzao's package too but we had to make a lot of improvements so we created this one. We made this package https://github.com/Konecty/meteor-multiple-instances-status to handle multiple server connections and handle user status if one of the servers dies. So, we can close this issue? Or are you experiencing this problem yet? |
Still experiencing it, but since it works fine for you, it's other packages I'm using that are causing the problems. Close if you want. |
No, let's try to solve this problem. You had tried to install this package https://github.com/Konecty/meteor-multiple-instances-status? |
Yeah. I have that and user presence installed. Plus meteorhacks:cluster.
|
Ok, let me know. |
I'm thinking I might be seeing this problem as well. After I did a push to my app the number of connections in Kadira showed around 40 and my in-app connection count using this package was around 20. |
to reproduce:
load 2 windows with users logged in. 2 users are now showing as online
change some server side code. the server will now refresh.
check how many users are online. it will be 0. hit refresh again on one of the clients and itll go up to 1.
The text was updated successfully, but these errors were encountered: