Update PhoneInputDialog UI/UX: tabs, quick dial dropdown, footer dial button - #4630
Draft
stephenhand with Copilot wants to merge 55 commits into
Draft
Update PhoneInputDialog UI/UX: tabs, quick dial dropdown, footer dial button#4630stephenhand with Copilot wants to merge 55 commits into
stephenhand with Copilot wants to merge 55 commits into
Conversation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
…nt/AsyncIterable pattern Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
…basic-sms-test-methods
- Add SMS channel to twilio-iac/helplines/e2e/development.hcl using the messaging-lex-v3-blocking-lambda.tftpl template (same as aselo_webchat) and an empty contact_identity (conversations address managed separately) - Create twilio-iac/helplines/e2e/files/additional.configure.tf that uses a Twilio data source to look up the only phone number attached to the account at apply time and creates the SMS conversations address linked to the SMS studio flow - Guard twilio_conversations_configuration_addresses_v1 in channels/v1/main.tf so channels with an empty contact_identity skip automatic address creation (allowing helpline-specific additional.tf to manage it instead) Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
…basic-sms-test-methods
- Replace dynamic phone number lookup with hardcoded +12607821891 in both development.hcl and additional.configure.tf - Remove data source that previously scraped the phone number from the Twilio account - Set contact_identity to the hardcoded phone number in the SMS channel configuration Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
The twilio-iac/terraform-modules/channels/v1/main.tf module now automatically creates the SMS conversations address since contact_identity is no longer empty. The separate additional.configure.tf file is no longer needed. Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
- Add voice channel using voice-no-chatbot-operating-hours-blocking-lambda template - Use the same phone number (+12607821891) as the SMS channel - Include voice_ivr_greeting_message, voice_ivr_blocked_message, and voice_ivr_language - Follows established patterns used in other helplines for voice configurations Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
… CHI-3711-voice_test_poc
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
…nt/AsyncIterable pattern Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
- Add SMS channel to twilio-iac/helplines/e2e/development.hcl using the messaging-lex-v3-blocking-lambda.tftpl template (same as aselo_webchat) and an empty contact_identity (conversations address managed separately) - Create twilio-iac/helplines/e2e/files/additional.configure.tf that uses a Twilio data source to look up the only phone number attached to the account at apply time and creates the SMS conversations address linked to the SMS studio flow - Guard twilio_conversations_configuration_addresses_v1 in channels/v1/main.tf so channels with an empty contact_identity skip automatic address creation (allowing helpline-specific additional.tf to manage it instead) Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
- Add voice channel using voice-no-chatbot-operating-hours-blocking-lambda template - Use the same phone number (+12607821891) as the SMS channel - Include voice_ivr_greeting_message, voice_ivr_blocked_message, and voice_ivr_language - Follows established patterns used in other helplines for voice configurations Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
…I-3711-voice_test_poc # Conflicts: # e2e-tests/tests/offlineContact.spec.ts
… CHI-3711-voice_test_poc
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…I-3711-voice_test_poc
CHI-3711: POC for voice and sms E2E tests - CI only
…button Co-authored-by: stephenhand <1694716+stephenhand@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update PhoneInputDialog UI/UX per requirements
Update PhoneInputDialog UI/UX: tabs, quick dial dropdown, footer dial button
Aug 21, 2026
stephenhand
changed the base branch from
master
to
CHI-3946-quick_dial_with_config
August 21, 2026 14:47
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
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.
Replaces the temporary POC-era
PhoneInputDialogwith a redesigned popover matching the approved design: conditional tabs, a quick dial dropdown, full-width manual entry, a footer-anchored primary Dial button, and auto-close on connect.Changes
PhoneInputDialog.tsxStyledTabs:Quick Dial/Enter Numbertabs render only when bothquickDialOptions(≥1) andenableManualDialare configured; otherwise the single available mode renders tab-free<select>pre-populated from config, defaults to first option, formatted asnumber | labelPrimaryButtonwithbuttonSize="small", moved to footer (right-aligned)#606B85, text "Example: +1 123-456-7890"<DialogArrow />(triangular pointer)styles.tsDialogArrowandFleximportPhoneDialogContent,PhoneDialogFooter,PhoneNumberInput,QuickDialSelect,HelpTexthrmConfig.tsQuickDialOptiontype ({ number, label })conferencing_quick_dial_options(array, default[]) andconferencing_enable_manual_dial(boolean, defaulttrue) from service configuration attributesConferencePanel.tsxquickDialOptionsandenableManualDialtoPhoneInputDialogcallStatus === 'in-progress'Translations
Conference-QuickDialTab,Conference-EnterNumberTab,Conference-PhoneNumberExample,Conference-QuickDialSelectLabelto all 6 translation files (en, en-CA, en-GB, es, fr, th)Verification steps
conferencing_quick_dial_optionsin service config attributes (e.g.[{"number": "988", "label": "Suicide and Crisis Lifeline"}]) andconferencing_enable_manual_dial: true— verify both tabs appear and the quick dial dropdown pre-selects the first entry; Dial button active immediatelyconferencing_enable_manual_dial: false— verify only Quick Dial renders with no tabsconferencing_quick_dial_options: []— verify only Enter Number renders with no tabs; Dial button disabled until input is non-emptyChecklist
Other Related Issues
None
AFTER YOU MERGE
You are responsible for ensuring the above steps are completed. If you move a ticket into QA without advising what version to test, the QA team will assume the latest tag has the changes. If it does not, the following confusion is on you! :-P