Skip to content

Conversation

@parfeon
Copy link
Contributor

@parfeon parfeon commented Aug 30, 2023

feat(event_engine): subscribe EE works with presence EE

On subscription changes and restore, the subscribe event engine should inform the presence event engine about such events so user_id presence will be properly updated.

feat(subscribe): add missing disconnect and reconnect methods

Add method which lets users temporarily stop real-time updates processing and restore when it will be needed.

On subscription changes and restore subscribe event engine should inform presence event engine
about such events so `user_id` presence will be properly updated.

feat(subscribe): add missing `disconnect` and `reconnect` methods

Add method which let users temporarily stop real-time updates processing and restore when it
will be needed.
@parfeon parfeon added priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: feature This PR contains new feature. labels Aug 30, 2023
@parfeon parfeon requested a review from Xavrax as a code owner August 30, 2023 00:30
@parfeon parfeon self-assigned this Aug 30, 2023
@Xavrax Xavrax added priority: high This PR should be reviewed ASAP. and removed priority: medium This PR should be reviewed after all high priority PRs. labels Aug 30, 2023
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 (except 1 thing)

parfeon and others added 4 commits August 30, 2023 15:38
`unsubscribe_all` will go through `Subscription` instances and force waker to schedule their
polling and return `Poll::Ready(None)` from `poll_next`.

test(contract): temporarily disable the presence event engine
@Xavrax Xavrax merged commit 3483fb6 into master Aug 30, 2023
@Xavrax Xavrax deleted the CLEN-1465/feat/subscribe-presence-ee-cooperate branch August 30, 2023 13:43
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.

2 participants