FE: Update VAMC Billing and Insurance Phone number to work with new fields

laflannery commented 5 months ago

Background

In #17861 the Phone number field was updated to use the Phone paragraph type field (instead of the Telephone number field that it previously was). This means the FE query needs to be updated to pull the data properly from these new fields and then display on the FE.

Description

The Phone paragraph type has more fields than what we previously had so there are more conditions to account for:

Type:
- Phone: standard component, no additional props
- Fax: the not-clickable prop in the component should be used
- SMS: the sms prop in the component should be used
- TTY: the tty prop should in the component be used
Phone number: This is the info that would go into the contact prop of the component
Extension number: This is the info that would go into the ext prop of the component
Label:
- This is hidden from Drupal and we will not be changing the current label that is used on the FE

Acceptance Criteria

[ ] If type Fax is selected, number is not clickable
[ ] If type SMS is selected, number uses the sms prop
[ ] If type TTY is selected, number shows "TTY"
[ ] Exts are displayed and parsed correctly
[ ] The existing hardcoded label displays
[ ] Requires accessibility review

### Testing & Launch
- [ ] The CMS PR from 17861 will need to be referenced in this PR
- [ ] Once the Content Build PR is ready, integrated testing of both PRs should happen on Tugboat
- [ ] Keep Content Build PR in Pending Integration until Drupal PR merges & deploys
- [ ] More launch details in Drupal ticket:
- [ ] https://github.com/department-of-veterans-affairs/va.gov-cms/issues/17861

davidmpickett commented 2 months ago

Hey team! Please add your planning poker estimate with Zenhub @chriskim2311 @eselkin @randimays

randimays commented 2 months ago

Need @eselkin to confirm, but these 3 phone number tickets (#17865, #17867 and this ticket), share a common phone-formatting utility. We should only need to update it once and then test for all.

davidmpickett commented 2 months ago

Closing in favor of an integrated full stack story #17861

department-of-veterans-affairs / va.gov-cms