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

mute all #1

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

mute all #1

wants to merge 7 commits into from

Conversation

OmriShraga
Copy link

added force mute and force mute all

lib/muteModeration.js Outdated Show resolved Hide resolved
lib/muteModeration.js Outdated Show resolved Hide resolved
lib/opentok.js Outdated Show resolved Hide resolved
lib/opentok.js Outdated Show resolved Hide resolved
lib/opentok.js Outdated Show resolved Hide resolved
lib/errors.js Show resolved Hide resolved
lib/muteModeration.js Show resolved Hide resolved
lib/muteModeration.js Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@TamirVonage TamirVonage removed their request for review March 1, 2021 08:28
Comment on lines +810 to +811
* @param callback {Function} The function to call upon completing the operation. Two arguments
* are passed to the function:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the second argument?

@@ -420,6 +420,37 @@ opentok.forceDisconnect(sessionId, connectionId, function (error) {
This is the server-side equivalent to the forceDisconnect() method in OpenTok.js:
<https://www.tokbox.com/developer/guides/moderation/js/#force_disconnect>.

### Force Mute a participant

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Force Mute a participant
### Force a publisher to mute audio

Copy link

@jeffswartz jeffswartz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a PR to this PR to add docs edits: #2. @OmriShraga , please review and merge in if it looks good. (I was not able to commit directly to your PR.)

Comment on lines +834 to +835
* @param callback {Function} The function to call upon completing the operation. Two arguments
* are passed to the function:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two arguments?

}, callback);
};

handleForceMute = function (config, sessionId, streamId, excludedStreamIds = null, callback) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting error here. The codebase is not using ES6 (and default parameters). And there are other linting errors. Please run npm run lint.

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.

5 participants