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

Fix cursor reset issue and cancelled effects spawning #184

Merged
merged 10 commits into from
Feb 8, 2024

Conversation

parfeon
Copy link
Contributor

@parfeon parfeon commented Jan 31, 2024

feat(subscription): add more types to create SubscriptionCursor from

Make it possible to create SubscriptionCursor from the string slice.

feat(subscription): add methods to manipulate subscriptions in set

Add add_subscriptions(..) and sub_subscriptions(..) to SubscriptionSet to make it possible in addition to sets manipulation use list of subscriptions.

fix(subscribe): fix cursor reset issue

Fix issue because of which cursor is not reset on Subscription and SubscriptionSet on unsubscribe.

fix(subscription): fix issue with cancelled effects

Fix issue because of which cancelled effects still asynchronously spawned for processing.

docs(inline): change client to pubnub in inline docs

refactor(subscribe): add subscription token validation

Added a method to validate the provided subscription token to conform to PubNub time token requirements with precision.

refactor(examples): refactor subscribe example

Separate subscribe example into two to show separately subscribe feature and presence state maintenance with subscribe.

refactor(examples): refactor `subscribe` example

Separate `subscribe` example into two to show separately `subscribe` feature and `presence state`
maintenance with subscribe.
Fix issue because of which `cursor` is not reset on `Subscription` and `SubscriptionSet`
on unsubscribe.
@parfeon parfeon added priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: fix This PR contains fixes to existing features. labels Jan 31, 2024
@parfeon parfeon self-assigned this Jan 31, 2024
parfeon and others added 4 commits February 5, 2024 10:37
Added a method to validate the provided subscription token to conform to PubNub time token
requirements with precision.

refactor(subscription): split `subscribe` method

Split the `subscribe` method into two: `subscribe` and `subscribe_with_timetoken`.
Fix issue because of which cancelled effects still asynchronously spawned for processing.

feat(subscription): add more types to create `SubscriptionCursor` from

Make it possible to crate `SubscriptionCursor` from the string slice.

feat(subscription): add methods to manipulate subscriptions in set

Add `add_subscriptions(..)` and `sub_subscriptions(..)` to `SubscriptionSet` to make it possible in
addition to sets manipulation use list of subscriptions.
@parfeon parfeon changed the title Fix cursor reset issue Fix cursor reset issue and cancelled effects spawning Feb 6, 2024
Copy link
Contributor

@Xavrax Xavrax left a comment

Choose a reason for hiding this comment

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

LGTM! Great work!

Only one proposal but it can be done in other PR (if we want to).

examples/subscribe.rs Show resolved Hide resolved
examples/subscribe.rs Show resolved Hide resolved
@parfeon
Copy link
Contributor Author

parfeon commented Feb 7, 2024

@pubnub-release-bot release

@parfeon parfeon merged commit dab433d into master Feb 8, 2024
9 checks passed
@parfeon parfeon deleted the refactor/update-docs-examples branch February 8, 2024 09:02
@pubnub-release-bot
Copy link

🚀 Release successfully completed 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium This PR should be reviewed after all high priority PRs. 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.

None yet

4 participants