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

Subscribe / presence stateless methods #97

Closed
wants to merge 4 commits into from

Conversation

parfeon
Copy link
Contributor

@parfeon parfeon commented Feb 8, 2022

feat: subscribe / presence stateless methods

Add stateless methods for subscribe / presence API.

Add stateless methods for subscribe / presence API.
Fix arguments which is passed to `SubscribeRouter` in `receiveMessages`.
Add flag which will allow to enable beta version of subscribe loop.
@parfeon parfeon added priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: refactor This PR contains refactored existing features. labels Feb 8, 2022
@parfeon parfeon requested a review from CraigLn February 8, 2022 14:20
@parfeon parfeon self-assigned this Feb 8, 2022
Copy link

@kleewho kleewho left a comment

Choose a reason for hiding this comment

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

What about setPresenceState utility?

@@ -33,7 +33,7 @@ public class SubscriptionSession {

public let uuid = UUID()
let longPollingSession: SessionReplaceable
internal(set) var configuration: SubscriptionConfiguration
var configuration: SubscriptionConfiguration
Copy link

Choose a reason for hiding this comment

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

Does it have to change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kleewho this one changed in master already.

@parfeon
Copy link
Contributor Author

parfeon commented Feb 8, 2022

@kleewho

What about setPresenceState utility?

Swift SDK already have setPresence. So I'm kind of confusing in why we need to create dupe.

@CraigLn
Copy link
Contributor

CraigLn commented Jan 9, 2023

@parfeon I'm just trying to close out PRs that I'm attached to. Is this PR still valid?

@parfeon
Copy link
Contributor Author

parfeon commented Jan 9, 2023

Yep. This one still valid. Waiting when there will be decision to make release.
Will send it to draft so far.

@parfeon
Copy link
Contributor Author

parfeon commented Aug 2, 2023

This has been done in a different way.

@parfeon parfeon closed this Aug 2, 2023
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: refactor This PR contains refactored existing features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants