Skip to content

fix(cc-components): show dialed number instead of entrypoint for outdial calls#657

Open
bhabalan wants to merge 2 commits intowebex:nextfrom
bhabalan:CAI-7359
Open

fix(cc-components): show dialed number instead of entrypoint for outdial calls#657
bhabalan wants to merge 2 commits intowebex:nextfrom
bhabalan:CAI-7359

Conversation

@bhabalan
Copy link
Contributor

COMPLETES

https://jira-eng-sjc12.cisco.com/jira/browse/CAI-7359

This pull request addresses

When making an outdial call via the dialpad, the incoming task, task list, and active task widgets incorrectly display the entrypoint number (ANI) instead of the actual dialed number (DN). This affects all three task display surfaces: IncomingTask, TaskList, and CallControlCAD.

by making the following changes

  • Modified extractIncomingTaskData in incoming-task.utils.tsx to check outboundType === 'OUTDIAL' and use dn (dialed number) instead of ani for the title
  • Applied the same fix to extractTaskListItemData in task-list.utils.ts for the task list display
  • Updated CallControlCAD component to show dn as the primary caller identifier for outdial calls
  • Falls back to ani if dn is not available, maintaining backward compatibility
  • Added 6 unit tests covering outdial title display, fallback behavior, and non-regression for inbound calls

Change Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Tooling change
  • Internal code refactor

The following scenarios were tested

  • The testing is done with the amplify link
  • Unit tests added/updated and passing (638 tests, all passing)
  • Outdial telephony task shows dialed number (dn) as title
  • Outdial task falls back to ANI when dn is unavailable
  • Inbound telephony tasks continue to show ANI as title (non-regression)

The GAI Coding Policy And Copyright Annotation Best Practices

  • GAI was not used (or, no additional notation is required)
  • Code was generated entirely by GAI
  • GAI was used to create a draft that was subsequently customized or modified
  • Coder created a draft manually that was non-substantively modified by GAI (e.g., refactoring was performed by GAI on manually written code)
  • Tool used for AI assistance (GitHub Copilot / Other - specify)
    • Github Copilot
    • Other - Claude Code
  • This PR is related to
    • Feature
    • Defect fix
    • Tech Debt
    • Automation

Checklist before merging

  • I have not skipped any automated checks
  • All existing and new tests passed
  • I have updated the testing document
  • I have tested the functionality with amplify link

Make sure to have followed the contributing guidelines before submitting.

…ial calls

When making an outdial call via dialpad, the incoming task, task list, and
active task were incorrectly showing the entrypoint number (ANI) instead of
the actual dialed number (DN). This fix checks for outboundType === 'OUTDIAL'
and uses the DN field from callAssociatedDetails for the title display.

CAI-7359
@bhabalan bhabalan marked this pull request as ready for review March 18, 2026 02:33
@bhabalan bhabalan requested a review from a team as a code owner March 18, 2026 02:33
});
});

describe('Outdial tasks', () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add more comprehensive tests

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added 8 comprehensive test cases (4 for IncomingTask, 4 for TaskList) covering:

  • Empty dn string fallback to ani
  • CALLBACK outboundType correctly uses ani (not dn)
  • Social media outdial tasks use customerName
  • Non-browser outdial button states (Ringing..., disabled accept)

Also fixed 3 prettier lint errors in source files. All 646 tests passing.

@bhabalan bhabalan added the validated Indicates that the PR is ready for actions label Mar 18, 2026
@aws-amplify-us-east-2
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-657.d1b38q61t1z947.amplifyapp.com

…lint errors

Add 4 new test cases each for IncomingTask and TaskList outdial scenarios:
- Empty dn string fallback to ani
- CALLBACK outboundType uses ani (not dn)
- Social media outdial uses customerName
- Non-browser outdial button states

Fix prettier formatting in source files.

CAI-7359
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

validated Indicates that the PR is ready for actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant