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

Add meeting conference numbers info to stateful client #4576

Merged
merged 13 commits into from
May 29, 2024

Conversation

rzdor
Copy link
Contributor

@rzdor rzdor commented May 3, 2024

Meeting conference numbers info will contain number and conference code to add ability to make pstn calls in Teams meetings.

This information could be found in this popup in Teams
image

rzdor and others added 2 commits May 3, 2024 11:22
Signed-off-by: Ruslan Zdor <99833343+rzdor@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented May 3, 2024

Copy link
Contributor

github-actions bot commented May 3, 2024

Copy link
Contributor

github-actions bot commented May 3, 2024

Copy link
Contributor

github-actions bot commented May 3, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6177499
  • Base size: 6176804
  • Diff size: 695

Copy link
Contributor

github-actions bot commented May 3, 2024

Chat bundle size is not changed.

  • Current size: 2061928
  • Base size: 2061928
  • Diff size: 0

Copy link
Contributor

github-actions bot commented May 3, 2024

Calling bundle size is increased❗.

  • Current size: 4852884
  • Base size: 4852189
  • Diff size: 695

Copy link
Contributor

github-actions bot commented May 8, 2024

Copy link
Contributor

github-actions bot commented May 9, 2024

Copy link
Contributor

github-actions bot commented May 9, 2024

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

Lines Statements Functions Branches
Base 25999 / 40683
63.9%
25999 / 40683
63.9%
710 / 1268
55.99%
2067 / 3261
63.38%
Current 25998 / 40682
63.9%
25998 / 40682
63.9%
710 / 1268
55.99%
2049 / 3251
63.02%
Diff -1 / -1
0%
-1 / -1
0%
0 / 0
0%
-18 / -10
-0.36%

Copy link
Contributor

github-actions bot commented May 9, 2024

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

Lines Statements Functions Branches
Base 50254 / 80897
62.12%
50254 / 80897
62.12%
1028 / 2283
45.02%
2916 / 4811
60.61%
Current 50277 / 80942
62.11%
50277 / 80942
62.11%
1029 / 2285
45.03%
2941 / 4824
60.96%
Diff 23 / 45
-0.01%
23 / 45
-0.01%
1 / 2
0.01%
25 / 13
0.35%

@rzdor rzdor removed the in progress label May 9, 2024
Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

info: callInfo
info: callInfo,
/* @conditional-compile-remove(teams-meeting-conference) */
teamsMeetingConference: undefined
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this defaulting to undefined in the case that there is no conference coordinates? like in the case that this is a ACS call?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, meeting conference will contain phone information and it is available only in meeting scenario

@rzdor rzdor enabled auto-merge (squash) May 29, 2024 18:19
Copy link
Contributor

@rzdor rzdor added the update_snapshots Set this label to request automated update of UI snapshots label May 29, 2024
@github-actions github-actions bot removed the update_snapshots Set this label to request automated update of UI snapshots label May 29, 2024
@rzdor rzdor merged commit 0660708 into main May 29, 2024
62 checks passed
@rzdor rzdor deleted the ruslanz/add-meeting-coordinates branch May 29, 2024 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants