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

[MM-44651] Implement MaxCallParticipants config setting #6334

Merged
merged 2 commits into from
Jun 3, 2022
Merged

Conversation

streamer45
Copy link
Contributor

Summary

PR makes the necessary changes to support the new MaxCallParticipants config setting.

@cpoile Again I tried to simplify as much as possible but will need some help testing this to make sure I haven't missed some edge case.

Ticket Link

https://mattermost.atlassian.net/browse/MM-44651

Checklist

  • Added or updated unit tests (required for all new features)

Related PRs

mattermost/mattermost-plugin-calls#93

Release Note

Added support for the new MaxCallParticipants config setting

@streamer45 streamer45 added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Jun 2, 2022
@streamer45 streamer45 requested review from cpoile and enahum June 2, 2022 11:48
@streamer45 streamer45 self-assigned this Jun 2, 2022
@streamer45
Copy link
Contributor Author

@amyblais This is the last PR we are planning to include in 1.53

@amyblais amyblais added the CherryPick/Approved Meant for the quality or patch release tracked in the milestone label Jun 2, 2022
@amyblais amyblais added this to the v1.53.0 milestone Jun 2, 2022
Copy link
Member

@cpoile cpoile left a comment

Choose a reason for hiding this comment

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

Looks good, I'll do QA on it.

@cpoile cpoile removed the 2: Dev Review Requires review by a core commiter label Jun 2, 2022
@cpoile
Copy link
Member

cpoile commented Jun 2, 2022

From what I could see, given the issues with testing mattermost/mattermost-plugin-calls#93 , this should work as expected. Let's ship it.

@cpoile cpoile removed the 3: QA Review Requires review by a QA tester label Jun 2, 2022
@cpoile cpoile self-requested a review June 2, 2022 22:20
@amyblais amyblais added the 4: Reviews Complete All reviewers have approved the pull request label Jun 2, 2022
@streamer45 streamer45 merged commit bb655c8 into master Jun 3, 2022
@mattermod
Copy link
Contributor

Cherry pick is scheduled.

@streamer45 streamer45 deleted the MM-44651 branch June 3, 2022 06:18
mattermost-build pushed a commit to mattermost-build/mattermost-mobile that referenced this pull request Jun 3, 2022
)

* Implement MaxCallParticipants config setting

* Add test

(cherry picked from commit bb655c8)
@mattermod mattermod added CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Jun 3, 2022
streamer45 added a commit that referenced this pull request Jun 3, 2022
* Implement MaxCallParticipants config setting

* Add test

(cherry picked from commit bb655c8)

Co-authored-by: Claudio Costa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants