feat(cc-widgets): multi party conference - #539
Merged
Merged
Conversation
rsarika
changed the title
rsarika
added
validated
Shreyas281299
previously requested changes
Oct 23, 2025
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
rsarika
added
the
run_e2e
adhmenon
reviewed
Nov 4, 2025
adhmenon
left a comment
adhmenon
left a comment
Contributor
There was a problem hiding this comment.
Have not reviewed tests but core code is looking good. Flows are all fine.
Have few doubts - but other than that, let us test once more and then the PR should be alright.
rarajes2
requested changes
Nov 5, 2025
rarajes2
left a comment
rarajes2
left a comment
Contributor
There was a problem hiding this comment.
Have done first round of review, will require multiple rounds of review as the changes are big.
| onDialNumberSelect: ((dialNumber: string, allowParticipantsToInteract: boolean) => void) | undefined, | ||
| onEntryPointSelect: | ||
| | ((entryPointId: string, entryPointName: string, allowParticipantsToInteract: boolean) => void) | ||
| | undefined |
Contributor
There was a problem hiding this comment.
Use the ? with variable instead of defining type undefined like this. Applicable everywhere
rsarika
dismissed
Shreyas281299’s stale review
November 6, 2025 05:26
as rajesh and adhwaith reviewing this now.
rarajes2
approved these changes
Nov 6, 2025
rarajes2
left a comment
rarajes2
left a comment
Contributor
There was a problem hiding this comment.
Few minor comments, rest looks good to me. We should create a ticket for corner cases that we are going to handle in next ticket.
Please address last few comments and create JIRA before merging the PR.
adhmenon
approved these changes
Nov 6, 2025
github-actions Bot
pushed a commit
that referenced
this pull request
Nov 11, 2025
# [1.28.0-ccwidgets.123](v1.28.0-ccwidgets.122...v1.28.0-ccwidgets.123) (2025-11-11) ### Features * **cc-widgets:** multi party conference ([#539](#539)) ([f2a23d4](f2a23d4))
Contributor
|
🎉 This PR is included in version 1.28.0-ccwidgets.123 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
COMPLETES #https://jira-eng-sjc12.cisco.com/jira/browse/CAI-7183
This pull request addresses
Implemented the Multi-Party Conference feature supporting more than three participants, along with the Conference Transfer functionality.
Conference Transfer Behavior:
An agent can initiate a transfer only during a consultation.
Scenario 1: If Agent A1 and Agent A2 are on a consultation while the customer is on hold, upon transfer, A1 leaves and A2 takes ownership of the call.
Scenario 2: If Agent A1 and Agent A3 are on a consultation while the customer and Agent A2 are already in the main conference, upon transfer, A1 leaves, A2 becomes the new owner, and A3 joins the main conference with A2 and the customer.
by making the following changes
Added new buttons for Conference and Exit Conference and added handling of those buttons
Change Type
The following scenarios were tested
< ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >
The GAI Coding Policy And Copyright Annotation Best Practices
Checklist before merging