-
Notifications
You must be signed in to change notification settings - Fork 16
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
Notify of disconnect #8
Comments
+1 something like this would be great, and easy to implement import { SmartDisconnect } from 'meteor/mixmax:smart-disconnect'
SmartDisconnect.on('disconnect', () => {
alert('You have been disconnected because you weren\'t viewing the page');
});
SmartDisconnect.on('reconnect', () => {
alert('You have been reconnected');
}); |
Providing a hook may be better with alert in case you decide to get fancy with bert pack of notifications. I don't belief it automatically reconnects you, so using a menu with button for reconnecting may be better. |
It also just occured to me that it should probably not enable for development environment. |
|
notifying of reconnect would be helpful for the sake of lag, while disconnect message then is somewhat pointless then? |
actually, it looks like perhaps this can be covered by simply listening to the reactive data source |
sounds good - file a PR |
Would be cool to have ui with the logic to have user know that he's disconnected.
The text was updated successfully, but these errors were encountered: