Skip to content

[Bug fix] Turn off camera when going on hold#3926

Merged
dmceachernmsft merged 5 commits intomainfrom
dmceachernmsft/hold-camera-fix
Dec 18, 2023
Merged

[Bug fix] Turn off camera when going on hold#3926
dmceachernmsft merged 5 commits intomainfrom
dmceachernmsft/hold-camera-fix

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

Add the turning off of the local video when the local user goes on hold

Why

For privacy reasons we are turning off the camera when the call goes on hold

https://skype.visualstudio.com/SPOOL/_workitems/edit/3524707

How Tested

Validated locally

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) December 18, 2023 21:24
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 18, 2023

CallWithChat bundle size is increased❗.

  • Current size: 6433804
  • Base size: 6433674
  • Diff size: 130

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 18, 2023

Calling bundle size is increased❗.

  • Current size: 5613844
  • Base size: 5613714
  • Diff size: 130

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 18, 2023

Chat bundle size is increased❗.

  • Current size: 1533877
  • Base size: 1533876
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 18, 2023

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 43883 / 70268
62.45%
43883 / 70268
62.45%
906 / 1971
45.96%
2608 / 4205
62.02%
Current 43866 / 70274
62.42%
43866 / 70274
62.42%
906 / 1971
45.96%
2601 / 4199
61.94%
Diff -17 / 6
-0.03%
-17 / 6
-0.03%
0 / 0
0%
-7 / -6
-0.08%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 18, 2023

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 21206 / 33825
62.69%
21206 / 33825
62.69%
579 / 1011
57.27%
1692 / 2694
62.8%
Current 21206 / 33825
62.69%
21206 / 33825
62.69%
579 / 1011
57.27%
1692 / 2694
62.8%
Diff 0 / 0
0%
0 / 0
0%
0 / 0
0%
0 / 0
0%

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the composite UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@dmceachernmsft dmceachernmsft added the update_snapshots Set this label to request automated update of UI snapshots label Dec 18, 2023
@github-actions github-actions Bot removed the update_snapshots Set this label to request automated update of UI snapshots label Dec 18, 2023
@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit f5d4aeb into main Dec 18, 2023
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/hold-camera-fix branch December 18, 2023 23:55
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.

3 participants