Skip to content

Conversation

@parfeon
Copy link
Contributor

@parfeon parfeon commented Mar 8, 2025

feat(subscription-set): subscription set will subscribe added entities

SubscriptionSet will subscribe / unsubscribe added / removed Subscription or SubscriptionSet objects if the set itself already subscribed.

BREAKING CHANGES: No need to call subscribe() / unsubscribe() for Subscription and SubscriptionSet added into / removed from already subscribed SubscriptionSet

fix(shared-worker): fix subscribe throttle

Fix issue because of which throttle didn't consider difference in client settings (throttled only by user ID and subscribe key, which is not enough).

fix(smart-heartbeat): fix smart heartbeat usage

With the fix, smart heartbeat as feature has been added to the SDK, and it is disabled by default.

`SubscriptionSet` will subscribe / unsubscribe added / removed `Subscription` or `SubscriptionSet`
objects if the set itself already subscribed.

fix(shared-worker): fix subscribe throttle

Fix issue because of which throttle didn't consider difference in client settings (throttled only
by user ID and subscribe key, which is not enough).

fix(smart-heartbeat): fix smart heartbeat usage

With the fix, smart heartbeat as feature has been added to the SDK, and it is disabled by default.
@parfeon parfeon added status: done This issue is considered resolved. priority: high This PR should be reviewed ASAP. type: fix This PR contains fixes to existing features. labels Mar 8, 2025
@parfeon parfeon self-assigned this Mar 8, 2025
Copy link
Contributor

@mohitpubnub mohitpubnub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@parfeon
Copy link
Contributor Author

parfeon commented Mar 10, 2025

@pubnub-release-bot release

@parfeon parfeon merged commit 61a55e7 into master Mar 10, 2025
6 of 7 checks passed
@parfeon parfeon deleted the fix/shared-worker-heartbeat-subscription-set branch March 10, 2025 10:17
@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: fix This PR contains fixes to existing features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants