fix(task-refactor): bug bash fixes#692
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3dee3c13ca
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const heldEntries = consultEntries.filter((media) => media.isHold === true); | ||
| const candidates = heldEntries.length > 0 ? heldEntries : consultEntries; |
There was a problem hiding this comment.
Stop preferring stale held consult legs
When a previous consult leg remains in interaction.media with isHold: true and the agent starts a new consult whose newer consult media is not held, this filter discards the newer leg before the recency comparison runs. In that re-consult scenario calculateConsultTimerData will keep using the stale held media and show Consult on Hold with the old timestamp even though the current consult is active; compare recency across all consult entries first, then derive hold state from the selected current leg.
Useful? React with 👍 / 👎.
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
COMPLETES #< INSERT LINK TO ISSUE >
This pull request addresses
Contact Center Call Control regressions in the task-refactor build around hold state, consult timers, and conference participant list visibility during multi-party flows.
Hold & consult timer issues
Conference participant list issues
by making the following changes
Hold logic (main CAD chip + timer)
Consult leg timer (Agent 1 initiator CAD)
Conference participant list
Quality
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
Make sure to have followed the contributing guidelines before submitting.