Skip to content

Fix reaction crash#4608

Merged
PorterNan merged 4 commits intomainfrom
jinan/fix-reaction-crash
May 15, 2024
Merged

Fix reaction crash#4608
PorterNan merged 4 commits intomainfrom
jinan/fix-reaction-crash

Conversation

@PorterNan
Copy link
Copy Markdown
Contributor

What

Fix reaction crash - when reactionResources is null or undefined, the reaction from another side will cause app crash

Bug link:
https://skype.visualstudio.com/SPOOL/_workitems/edit/3720693

Why

How Tested

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
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2024

Calling bundle size is increased❗.

  • Current size: 4892097
  • Base size: 4892088
  • Diff size: 9

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2024

Chat bundle size is not changed.

  • Current size: 2180378
  • Base size: 2180378
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6344470
  • Base size: 6344461
  • Diff size: 9

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2024

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

Lines Statements Functions Branches
Base 25430 / 39463
64.44%
25430 / 39463
64.44%
699 / 1230
56.82%
2008 / 3192
62.9%
Current 25335 / 39463
64.19%
25335 / 39463
64.19%
695 / 1223
56.82%
2000 / 3165
63.19%
Diff -95 / 0
-0.25%
-95 / 0
-0.25%
-4 / -7
0%
-8 / -27
0.29%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2024

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

Lines Statements Functions Branches
Base 49731 / 79394
62.63%
49731 / 79394
62.63%
1017 / 2249
45.22%
2896 / 4760
60.84%
Current 49486 / 79398
62.32%
49486 / 79398
62.32%
1013 / 2242
45.18%
2851 / 4698
60.68%
Diff -245 / 4
-0.31%
-245 / 4
-0.31%
-4 / -7
-0.04%
-45 / -62
-0.16%

@PorterNan PorterNan enabled auto-merge (squash) May 15, 2024 18:27
@github-actions
Copy link
Copy Markdown
Contributor

@PorterNan PorterNan merged commit d7dcb90 into main May 15, 2024
@PorterNan PorterNan deleted the jinan/fix-reaction-crash branch May 15, 2024 18:39
PorterNan added a commit that referenced this pull request May 17, 2024
* Fix crash bug for remote screen share
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