Skip to content
This repository has been archived by the owner on Aug 30, 2024. It is now read-only.

Registering VideoControllers bug #65

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

Conversation

leeprobert
Copy link

The ConferenceController RegisterVideoControllers would break out of the iteration of VideoControllers after it found the local player. This prevented any other remote players from having their VideoTrack assigned.

… local player

The ConferenceController was set to return after finding the VideoController that was local. This meant any remote participants AFTER this point did not get their VideoTrack assigned.
@dolby-io-open-source-tools
Copy link

Hi @leeprobert!
Thank you for your pull request and welcome to our community.
We require contributors to sign our Contributor License Agreement, and yours needs attention.

Process

In order for us to review and merge your suggested changes, please sign at https://contribute.dolby.io. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.
Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request check will be unlocked.
MSG1001

@leeprobert
Copy link
Author

I also added a public method to the ConferenceController to ensure you can register the delegate methods at any point and not just in the Start method.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant