Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VACMS-18281 Added character counters & help text to phone paragraph type #18542

Merged

Conversation

davidmpickett
Copy link
Contributor

@davidmpickett davidmpickett commented Jul 17, 2024

Description

Relates to #18281 & #15803

Testing done

Using the entity relationship diagram, I identified 4 different Drupal entities that currently use the Phone Paragraph type

Screenshot 2024-07-17 at 12 23 03 PM

  • VAMC System VA police Page -> Tested directly
  • Service Location paragraph -> Tested via VAMC Facility Health Service
  • Contact Information paragraph -> Tested via Resources and Support Detail Page
  • VAMC Facility Health Service node -> (A legacy field which is hidden in the UI, but hasn't been deleted yet)

Screenshots

Character counts updates without help text changes

Screenshot 2024-07-17 at 12 38 10 PM

Screenshot 2024-07-17 at 12 38 17 PM

Help Text changes without character count update

Screenshot 2024-07-17 at 2 22 21 PM

All together now

Screenshot 2024-07-17 at 2 36 09 PM

QA steps

What needs to be checked to prove this works?
What needs to be checked to prove it didn't break any related things?
What variations of circumstances (users, actions, values) need to be checked?

  1. Log in to tugboat as an Admin (e.g. [email protected])
  2. Add a new VAMC Facility Health Service
    • Under Service Location go to Appointments > Phone number for appointments > Add a phone number
    • Under Service Location go to Contact Info > Phone > Add a phone number
    • Validate that character counters behave as expected
    • Validate that help text shows up as expected
  3. Add a new VAMC System VA police Page
    • Under Contact phone number
    • Validate that character counters behave as expected
    • Validate that help text shows up as expected
  4. Add a new Resources & and Support detail page
    • Under Contact information > Add a phone number
    • Validate that character counters behave as expected
    • Validate that help text shows up as expected

Definition of Done

  • Automated tests have passed.
  • Acceptance Criteria in related issue are met.
  • Manual Code Review Approved.

Select Team for PR review

  • CMS Team
  • Public websites
  • Facilities
  • User support
  • Accelerated Publishing

@va-cms-bot va-cms-bot temporarily deployed to Tugboat July 17, 2024 16:38 Destroyed
@github-actions github-actions bot added the Facilities Facilities products (VAMC, Vet Center, etc) label Jul 17, 2024
@davidmpickett davidmpickett marked this pull request as ready for review July 17, 2024 17:40
@davidmpickett davidmpickett requested review from a team, thejordanwood and laflannery July 17, 2024 17:41
@va-cms-bot va-cms-bot temporarily deployed to Tugboat July 17, 2024 19:27 Destroyed
@davidmpickett
Copy link
Contributor Author

Updated so this will close two related issues instead of just 1

@davidmpickett davidmpickett changed the title VACMS-18281 Added character counter to phone paragraph type VACMS-18281 Added character countesr & help text to phone paragraph type Jul 17, 2024
@davidmpickett davidmpickett changed the title VACMS-18281 Added character countesr & help text to phone paragraph type VACMS-18281 Added character counters & help text to phone paragraph type Jul 17, 2024
thejordanwood
thejordanwood previously approved these changes Jul 17, 2024
Copy link

@thejordanwood thejordanwood left a comment

Choose a reason for hiding this comment

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

Looks great!

@laflannery
Copy link
Contributor

The help text does not appear to match what's in Figma:
image

@davidmpickett
Copy link
Contributor Author

The help text does not appear to match what's in Figma: image

Gah, I was bamboozled by a comment preview

Screenshot 2024-07-18 at 10 59 16 AM

Foolishly, I took it at it's word and did not dive deeper

Screenshot 2024-07-18 at 11 02 14 AM

I'll go fix that now

@va-cms-bot va-cms-bot temporarily deployed to Tugboat July 18, 2024 16:04 Destroyed
@thejordanwood thejordanwood self-requested a review July 18, 2024 17:08
Copy link

@thejordanwood thejordanwood left a comment

Choose a reason for hiding this comment

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

@laflannery You have the eagle eyes today, because apparently I'm blind as a bat.

@davidmpickett I've re-reviewed the new tugboat and can confirm that the phone number field now says "Use the format 123-456-7890." I also want to call out that the label field says "Enter what office..." instead of "Indicate what office..." like in the Figma file, but I actually prefer to use the word "enter" here.

@laflannery
Copy link
Contributor

If Jordan approves the change that works for me!

@va-cms-bot va-cms-bot temporarily deployed to Tugboat July 22, 2024 13:44 Destroyed
Copy link
Contributor

@omahane omahane left a comment

Choose a reason for hiding this comment

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

Everything checks out and looks good. Nice work.

@davidmpickett davidmpickett merged commit 0f6d694 into main Jul 22, 2024
29 checks passed
@davidmpickett davidmpickett deleted the VACMS-18281-add-character-counter-to-phone-paragraph branch July 22, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Facilities Facilities products (VAMC, Vet Center, etc) sitewide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants