Skip to content

[Chat] Updated sample to disable BYOS file sharing in Teams calls#4786

Merged
jpeng-ms merged 5 commits intomainfrom
john/hide-byos-for-teams-call
Jun 26, 2024
Merged

[Chat] Updated sample to disable BYOS file sharing in Teams calls#4786
jpeng-ms merged 5 commits intomainfrom
john/hide-byos-for-teams-call

Conversation

@jpeng-ms
Copy link
Copy Markdown
Member

@jpeng-ms jpeng-ms commented Jun 26, 2024

What

BYOS file sharing is only supported in ACS group calls, so we need to hide the attachment upload icon for Teams calls.

ACS call:
image
Teams call:
image

Why

improvement

How Tested

callwithchat sample app

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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 26, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6403112
  • Base size: 6403060
  • Diff size: 52

@github-actions
Copy link
Copy Markdown
Contributor

Calling bundle size is not changed.

  • Current size: 5036881
  • Base size: 5036881
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

Chat bundle size is not changed.

  • Current size: 2104679
  • Base size: 2104679
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 26, 2024

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

Lines Statements Functions Branches
Base 26245 / 41117
63.83%
26245 / 41117
63.83%
715 / 1284
55.68%
2095 / 3322
63.06%
Current 26244 / 41117
63.82%
26244 / 41117
63.82%
715 / 1284
55.68%
2103 / 3326
63.22%
Diff -1 / 0
-0.01%
-1 / 0
-0.01%
0 / 0
0%
8 / 4
0.16%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 26, 2024

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

Lines Statements Functions Branches
Base 51232 / 82930
61.77%
51232 / 82930
61.77%
1039 / 2329
44.61%
3012 / 4930
61.09%
Current 51239 / 82930
61.78%
51239 / 82930
61.78%
1039 / 2329
44.61%
3019 / 4932
61.21%
Diff 7 / 0
0.01%
7 / 0
0.01%
0 / 0
0%
7 / 2
0.12%

@jpeng-ms jpeng-ms enabled auto-merge (squash) June 26, 2024 18:06
@jpeng-ms jpeng-ms merged commit 38bb51f into main Jun 26, 2024
@jpeng-ms jpeng-ms deleted the john/hide-byos-for-teams-call branch June 26, 2024 18:20
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