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

EDGE-1027 removed unnecessary participants from subscription object #37

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mmcdonald-bw
Copy link
Contributor

The way the API works, if you only supply a session ID in the subscriptions argument, you will automatically be subscribed to all participants in a session. This makes supplying participant ids unnecessary in this case, which is why I have removed them.

@mmcdonald-bw mmcdonald-bw requested review from a team, ajrice6713 and ckoegel June 28, 2022 19:27
@ckoegel
Copy link
Contributor

ckoegel commented Jul 1, 2022

can you still specify participants and only be subscribed to those? If so, do we want this example to specify that it is subscribing to all participants and give another example for only a select participant/s?

@mmcdonald-bw mmcdonald-bw marked this pull request as draft July 7, 2022 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants