Skip to content

Fix acs room audio video enabled notifications#5563

Merged
fuyan2024 merged 9 commits intomainfrom
fuyan/hard-mute-acs-room-capability
Jan 16, 2025
Merged

Fix acs room audio video enabled notifications#5563
fuyan2024 merged 9 commits intomainfrom
fuyan/hard-mute-acs-room-capability

Conversation

@fuyan2024
Copy link
Copy Markdown
Member

@fuyan2024 fuyan2024 commented Jan 16, 2025

What

Why

How Tested

Acs user join a room as presenter:
Before:
image

After:
image

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@fuyan2024 fuyan2024 marked this pull request as ready for review January 16, 2025 05:28
@fuyan2024 fuyan2024 requested review from a team as code owners January 16, 2025 05:28
@fuyan2024 fuyan2024 added the update_snapshots Set this label to request automated update of UI snapshots label Jan 16, 2025
@github-actions github-actions Bot removed the update_snapshots Set this label to request automated update of UI snapshots label Jan 16, 2025
@fuyan2024 fuyan2024 changed the title Fix acs room capability issue. Fix acs room audio video enabled notifications Jan 16, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 16, 2025

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

Lines Statements Functions Branches
Base 27912 / 44309
62.99%
27912 / 44309
62.99%
782 / 1423
54.95%
2321 / 3697
62.78%
Current 27910 / 44311
62.98%
27910 / 44311
62.98%
782 / 1423
54.95%
2330 / 3706
62.87%
Diff -2 / 2
-0.01%
-2 / 2
-0.01%
0 / 0
0%
9 / 9
0.09%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 16, 2025

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

Lines Statements Functions Branches
Base 56800 / 93453
60.77%
56800 / 93453
60.77%
1156 / 2669
43.31%
3460 / 5716
60.53%
Current 56747 / 93466
60.71%
56747 / 93466
60.71%
1156 / 2669
43.31%
3481 / 5720
60.85%
Diff -53 / 13
-0.06%
-53 / 13
-0.06%
0 / 0
0%
21 / 4
0.32%

@fuyan2024 fuyan2024 requested review from a team as code owners January 16, 2025 17:26
@fuyan2024 fuyan2024 requested a review from jimchou-dev January 16, 2025 17:26
@github-actions
Copy link
Copy Markdown
Contributor

Calling bundle size is not changed.

  • Current size: 11767065
  • Base size: 11767065
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

CallWithChat bundle size is not changed.

  • Current size: 11767078
  • Base size: 11767078
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

Chat bundle size is not changed.

  • Current size: 1769573
  • Base size: 1769573
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

@fuyan2024 fuyan2024 merged commit 7cdaf40 into main Jan 16, 2025
@fuyan2024 fuyan2024 deleted the fuyan/hard-mute-acs-room-capability branch January 16, 2025 18:05
fuyan2024 added a commit that referenced this pull request Jan 16, 2025
fuyan2024 added a commit that referenced this pull request Jan 16, 2025
fuyan2024 added a commit that referenced this pull request Jan 16, 2025
fuyan2024 added a commit that referenced this pull request Jan 16, 2025
carocao-msft added a commit that referenced this pull request Jan 24, 2025
* stable-remove-beta-suffix version bump

* Collect CHANGELOG

* Create groom me file for groom changelog branch (#5549)

* Create groom me file for groom changelog branch

* groomc

* update

* Update CHANGELOG to consolidate PR referencesupdate

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* change (#5557)

* Bump package versions to beta.0 for next release

* Cherry pick PR: #5563

* revert previous commit

* Update

* update

---------

Signed-off-by: carocao-msft <96077406+carocao-msft@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: fuyan <fuyan@microsoft.com>
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.

4 participants