-
Notifications
You must be signed in to change notification settings - Fork 28
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
Conversation
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.
There was a problem hiding this 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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
Swift SDK already have |
@parfeon I'm just trying to close out PRs that I'm attached to. Is this PR still valid? |
Yep. This one still valid. Waiting when there will be decision to make release. |
This has been done in a different way. |
feat: subscribe / presence stateless methods
Add stateless methods for subscribe / presence API.