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

together mode reference update #5605

Merged
merged 5 commits into from
Feb 3, 2025
Merged

Conversation

cn0151
Copy link
Member

@cn0151 cn0151 commented Jan 31, 2025

What

Update together mode feature's sdk references

Why

How Tested

Notification Stack:
image

VideoGallery
image

Copy link
Contributor

Copy link
Contributor

@edwardlee-msft
Copy link
Contributor

Looks good to me, feel free to put it out of draft mode

Copy link
Contributor

@cn0151 cn0151 added the does not need changelog Changes that does not affect the published package in any way do not need changelog entry label Feb 3, 2025
Copy link
Contributor

github-actions bot commented Feb 3, 2025

Chat bundle size is not changed.

  • Current size: 1776975
  • Base size: 1776975
  • Diff size: 0

Copy link
Contributor

github-actions bot commented Feb 3, 2025

Calling bundle size is not changed.

  • Current size: 12401100
  • Base size: 12401100
  • Diff size: 0

Copy link
Contributor

github-actions bot commented Feb 3, 2025

CallWithChat bundle size is not changed.

  • Current size: 12401112
  • Base size: 12401112
  • Diff size: 0

Copy link
Contributor

github-actions bot commented Feb 3, 2025

Copy link
Contributor

github-actions bot commented Feb 3, 2025

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

Lines Statements Functions Branches
Base 27886 / 44338
62.89%
27886 / 44338
62.89%
782 / 1423
54.95%
2328 / 3701
62.9%
Current 27932 / 44338
62.99%
27932 / 44338
62.99%
782 / 1423
54.95%
2343 / 3713
63.1%
Diff 46 / 0
0.1%
46 / 0
0.1%
0 / 0
0%
15 / 12
0.2%

Copy link
Contributor

github-actions bot commented Feb 3, 2025

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

Lines Statements Functions Branches
Base 56916 / 93964
60.57%
56916 / 93964
60.57%
1161 / 2687
43.2%
3477 / 5733
60.64%
Current 56944 / 93990
60.58%
56944 / 93990
60.58%
1161 / 2687
43.2%
3439 / 5711
60.21%
Diff 28 / 26
0.01%
28 / 26
0.01%
0 / 0
0%
-38 / -22
-0.43%

@cn0151 cn0151 merged commit 0d02739 into main Feb 3, 2025
57 of 87 checks passed
@cn0151 cn0151 deleted the cnwankwo/update-together-mode-ref branch February 3, 2025 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
does not need changelog Changes that does not affect the published package in any way do not need changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants