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.

Sign up for GitHub

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

Jump to bottom

Remove heartbeat? #2

Open
dburles opened this issue Feb 1, 2014 · 5 comments
Open

Remove heartbeat? #2

dburles opened this issue Feb 1, 2014 · 5 comments
Labels
enhancement

Comments

@dburles
Copy link
Owner

dburles commented Feb 1, 2014

Hopefully it can be removed once meteor/meteor#1769 gets through.
@dburles
Copy link
Owner Author

dburles commented Jul 28, 2014

The heartbeat will most likely have to remain as the DDP heartbeat duration is too long to be suitable

@snshn
Copy link

snshn commented Dec 10, 2014

As far as I understand it's been fixed now. My counter gets updated with no delay on the client when I open/close new tabs. Please check to see if I'm right.

Thanks.

@dburles
Copy link
Owner Author

dburles commented Dec 11, 2014

Right so the underlying thing though is that the timeout internal to Meteor is much longer than our heartbeat, which isn't really ideal for the type of use this package is intended for.

@snshn
Copy link

snshn commented Dec 11, 2014

Good job.

@JorritPosthuma
Copy link

See here for Presence with heartbeat: https://github.com/3stack-software/meteor-presence

@dburles dburles changed the title Heartbeat Remove heartbeat? May 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dburles @JorritPosthuma @snshn