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

1.2.0 Feature: Realtime: Channel Modes #33

Merged
merged 2 commits into from
Sep 9, 2022

Conversation

QuintinWillison
Copy link
Contributor

@QuintinWillison QuintinWillison commented Aug 15, 2022

I could see no evidence that the Ruby SDK supports this.

One of a family of pull requests addressing #3.

@QuintinWillison QuintinWillison self-assigned this Aug 15, 2022
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix August 15, 2022 10:54 Inactive
@QuintinWillison QuintinWillison marked this pull request as ready for review August 15, 2022 10:55
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix August 16, 2022 06:23 Inactive
sdk.yaml Show resolved Hide resolved
sdk.yaml Outdated Show resolved Hide resolved
Base automatically changed from realtime-delta-subscription-enhancements to main September 9, 2022 08:22
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix September 9, 2022 08:23 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix September 9, 2022 08:32 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix September 9, 2022 08:35 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix September 9, 2022 08:36 Inactive
@QuintinWillison QuintinWillison marked this pull request as draft September 9, 2022 08:37
@QuintinWillison
Copy link
Contributor Author

I have made a mistake with Git merges. Trying to resolve.

… via options for a channel within a Realtime client instance.

I could see no evidence that the Ruby SDK supports this.
@github-actions github-actions bot temporarily deployed to staging/pull/33/matrix September 9, 2022 08:40 Inactive
@QuintinWillison QuintinWillison marked this pull request as ready for review September 9, 2022 08:41
@QuintinWillison
Copy link
Contributor Author

Having addressed @lmars' comment with help from @SimonWoolf, I'm going to merge this pull request without waiting for approval. It has been open for a while and there is a stack of pull requests ahead of it. We are likely to have another look at the entire work at the end, to double check that it congeals holistically, so the risk of merge without approval is low.

@QuintinWillison QuintinWillison merged commit 9a68f6f into main Sep 9, 2022
@QuintinWillison QuintinWillison deleted the realtime-channel-modes branch September 9, 2022 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants