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

Move heartbeat call into SharedWorker #433

Merged
merged 5 commits into from
Jan 30, 2025
Merged

Move heartbeat call into SharedWorker #433

merged 5 commits into from
Jan 30, 2025

Conversation

parfeon
Copy link
Contributor

@parfeon parfeon commented Jan 30, 2025

feat(shared-worker): move heartbeat call into SharedWorker

Pass heartbeat request through SharedWorker (if used) to optimize the number of requests for clients opened in few tabs and subscribed on same channels / groups list.

refactor(heartbeat): remove redundant heartbeat from unsubscribe

Don't send heartbeat request to unsubscribe.

Pass heartbeat request through `SharedWorker`` (if used) to optimize the number of requests for
clients opened in few tabs and subscribed on same channels / groups list.

refactor(heartbeat): remove redundant `heartbeat` from unsubscribe

Don't send `heartbeat` request on unsubscribe.
@parfeon parfeon added status: done This issue is considered resolved. priority: high This PR should be reviewed ASAP. type: feature This PR contains new feature. labels Jan 30, 2025
@parfeon parfeon self-assigned this Jan 30, 2025
@parfeon parfeon changed the title Move heartbeat call into SharedWorker Move heartbeat call into SharedWorker Jan 30, 2025
@parfeon
Copy link
Contributor Author

parfeon commented Jan 30, 2025

@pubnub-release-bot release

@parfeon parfeon merged commit 0d424f9 into master Jan 30, 2025
6 of 7 checks passed
@parfeon parfeon deleted the CLEN-2503 branch January 30, 2025 18:06
@pubnub-release-bot
Copy link
Contributor

🚀 Release successfully completed 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high This PR should be reviewed ASAP. status: done This issue is considered resolved. type: feature This PR contains new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants