Skip to content

[End Call Options] End call for everyone only for when the user is connected#4768

Merged
dmceachernmsft merged 6 commits intomainfrom
dmceachernmsft/end-call-ev-lobby
Jun 21, 2024
Merged

[End Call Options] End call for everyone only for when the user is connected#4768
dmceachernmsft merged 6 commits intomainfrom
dmceachernmsft/end-call-ev-lobby

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

Only allow the end call options for when the user is actively in the call

Why

The user should only be able to end the call if they are actively in the call so they can communicate to the others in the call they are ending it for everyone.

How Tested

Validated locally
image
image
image

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2024

Calling bundle size is increased❗.

  • Current size: 5035574
  • Base size: 5035538
  • Diff size: 36

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2024

Chat bundle size is increased❗.

  • Current size: 2099590
  • Base size: 2099589
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6396315
  • Base size: 6396280
  • Diff size: 35

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2024

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

Lines Statements Functions Branches
Base 26191 / 41117
63.69%
26191 / 41117
63.69%
715 / 1284
55.68%
2075 / 3303
62.82%
Current 26244 / 41117
63.82%
26244 / 41117
63.82%
715 / 1284
55.68%
2075 / 3310
62.68%
Diff 53 / 0
0.13%
53 / 0
0.13%
0 / 0
0%
0 / 7
-0.14%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2024

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

Lines Statements Functions Branches
Base 51214 / 82961
61.73%
51214 / 82961
61.73%
1039 / 2328
44.63%
3023 / 4936
61.24%
Current 51291 / 82964
61.82%
51291 / 82964
61.82%
1039 / 2328
44.63%
3027 / 4940
61.27%
Diff 77 / 3
0.09%
77 / 3
0.09%
0 / 0
0%
4 / 4
0.03%

@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) June 21, 2024 22:37
@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit 8ba171e into main Jun 21, 2024
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/end-call-ev-lobby branch June 21, 2024 22:49
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