-
Notifications
You must be signed in to change notification settings - Fork 221
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
Handle subscription peer add/remove events #374
Comments
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
Add events for subscription with `op` values of `add`, `remove`, and `update`. `peer_add` and `peer_remove` left for zulip#374.
These events will become relevant only in a future where we start tracking the data that they update, namely the list of other subscribers to each channel. If we do add such a data structure, we'll naturally go and handle the events that update it; that's the pattern for implementing new pieces of the Zulip data model in general. So I think there isn't an independent task to be tracked here, and isn't a need for this issue to track it. Instead, if at some point we decide to add a feature that would use that data, we'll file issues as needed to describe what that feature needs. |
Part of #135
https://zulip.com/api/get-events#subscription-peer_add
https://zulip.com/api/get-events#subscription-peer_remove
This issue is for handling when subscribers on a subscription are being modified.
subscription op:add
,subscription op:remove
andsubscription op:update
are being handled as part of #187The text was updated successfully, but these errors were encountered: