Skip to content

Fix runtime error when starting screenshare alone in a call#5240

Merged
mgamis-msft merged 5 commits intomainfrom
mgamis/fix-screenshare-when-alone-in-call
Oct 3, 2024
Merged

Fix runtime error when starting screenshare alone in a call#5240
mgamis-msft merged 5 commits intomainfrom
mgamis/fix-screenshare-when-alone-in-call

Conversation

@mgamis-msft
Copy link
Copy Markdown
Contributor

@mgamis-msft mgamis-msft commented Oct 3, 2024

What

Fix runtime error when starting screenshare alone in a call

Why

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

How Tested

Local calling sample test:
https://github.com/user-attachments/assets/5475e86a-1a98-4ae2-9694-e0cf58b10f9e

Screenshot of test:
Screenshot 2024-10-03 113308

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 Oct 3, 2024

CallWithChat bundle size is not changed.

  • Current size: 11750193
  • Base size: 11750193
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

Calling bundle size is not changed.

  • Current size: 11750179
  • Base size: 11750179
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

Chat bundle size is not changed.

  • Current size: 1756520
  • Base size: 1756520
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

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

Lines Statements Functions Branches
Base 26593 / 42860
62.04%
26593 / 42860
62.04%
727 / 1327
54.78%
2072 / 3339
62.05%
Current 25773 / 42860
60.13%
25773 / 42860
60.13%
701 / 1304
53.75%
1916 / 3063
62.55%
Diff -820 / 0
-1.91%
-820 / 0
-1.91%
-26 / -23
-1.03%
-156 / -276
0.5%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

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

Lines Statements Functions Branches
Base 53930 / 88128
61.19%
53930 / 88128
61.19%
1083 / 2472
43.81%
3189 / 5248
60.76%
Current 52613 / 88129
59.69%
52613 / 88129
59.69%
1057 / 2449
43.16%
2956 / 4923
60.04%
Diff -1317 / 1
-1.5%
-1317 / 1
-1.5%
-26 / -23
-0.65%
-233 / -325
-0.72%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

@mgamis-msft mgamis-msft enabled auto-merge (squash) October 3, 2024 20:53
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

@mgamis-msft mgamis-msft merged commit 5a704d8 into main Oct 3, 2024
@mgamis-msft mgamis-msft deleted the mgamis/fix-screenshare-when-alone-in-call branch October 3, 2024 21:04
mgamis-msft added a commit that referenced this pull request Oct 3, 2024
* Fix runtime error when starting screenshare alone in a call

* Change files

* add unit test

---------

Co-authored-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>
mgamis-msft added a commit that referenced this pull request Oct 3, 2024
…5244)

* Fix runtime error when starting screenshare alone in a call

* Change files

* add unit test

---------

Co-authored-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.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.

3 participants