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

Align Main mental health phone number with service contact info #16652

Closed
2 tasks done
mmiddaugh opened this issue Jan 3, 2024 · 15 comments
Closed
2 tasks done

Align Main mental health phone number with service contact info #16652

mmiddaugh opened this issue Jan 3, 2024 · 15 comments
Assignees
Labels
Facilities Facilities products (VAMC, Vet Center, etc) sitewide UX VA.gov frontend CMS team practice area VAMC CMS managed product owned by Facilities team

Comments

@mmiddaugh
Copy link
Contributor

mmiddaugh commented Jan 3, 2024

Description

CMS became the source of truth for Mental health phone numbers in #9717.
Mental health numbers appear in a number of places throughout the VAMC sites and in the Facility Locator.

  • The main number is displayed at the facility level in the Facility Locator search results and in the Locations and Contact information block on VAMC pages.
  • One or more phone numbers can be added at the Mental health service level. These numbers might be labeled Main, Contact, Appointment, or other.
  • Contact information for Care Coordinators are displayed on required Mental health care pages (which are linked from within the service accordion)

Problems to solve

  • If a Veteran refers to the health service accordion to find a number, will the best number be listed?
  • And if it is listed, does it have a clear, identifiable label?
  • Do the two numbers labeled as "Main" agree?
  • How might we align this information to remove confusion for Veterans?

System level mental health numbers are scoped separately in #16655

Current Examples to Avoid

Hot Springs VA Medical Center

Phone numbers displayed in Location and contact information area

  • Main phone: 605-745-2000
  • Mental health care: 605-347-2511

Within the Mental health service accordion

  • Main Phone: 605-745-2000 (appears twice)
  • Appointments: none listed

Cheyenne VA Medical Center

Phone numbers displayed in Location and contact information area

  • Main phone: 307-778-7550
  • Mental health care: 307-778-7349

Within the Mental health service accordion

  • Main Phone: 307-778-7550
  • Contact: 307-778-7550, ext. 7349
  • Contact: 307-920-3536
  • Appointments: 888-483-9127

Engineering notes

  • From discussion, logic is already in place.
  • FE change needed = look at service, if it's mental health, use the number specified by the facility
  • Could also make an improvement in Drupal to align UX with front end experience to help editors avoid duplication. If we don't make these changes now, we'll need to make the existing behavior clear in KB article to support editors.

Acceptance Criteria

  • The Mental health phone number listed under contact information should also appear as the Mental health phone number in the Mental health service accordion.
  • Use the telephone component with the aria described by label.
@mmiddaugh mmiddaugh added Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) VAMC CMS managed product owned by Facilities team labels Jan 3, 2024
@xiongjaneg xiongjaneg added the Needs refining Issue status label Jan 3, 2024
@xiongjaneg
Copy link
Contributor

Questions to be asked:

  • How do we prevent the editor from showing a number twice?

Front end:

  • What if the editor didn't check the box for Use main facility number, should we grab the mental health number anyway?
  • A separate ticket for Use main facility number, which is currently a front end function
  • To prevent duplicating a number, front end may have to add some logic to compare and prevent duplication - current state

@xiongjaneg xiongjaneg added the VA.gov frontend CMS team practice area label Jan 10, 2024
@xiongjaneg
Copy link
Contributor

xiongjaneg commented Jan 11, 2024

@xiongjaneg This is a front end change that would need change management.

@xiongjaneg xiongjaneg added UX and removed Drupal engineering CMS team practice area labels Jan 11, 2024
@xiongjaneg
Copy link
Contributor

@davidmpickett Please review for any related impact to/from #16655

@davidmpickett davidmpickett self-assigned this Jan 11, 2024
@mmiddaugh
Copy link
Contributor Author

FE change needed = look at service, if it's mental health, use the number specified by the facility
In addition to FE change, we could make an improvement in Drupal to align UX with front end experience to help editors avoid duplication. If we don't make these changes now, we'll need to make the existing behavior clear in KB article to support editors.

@xiongjaneg
Copy link
Contributor

@xiongjaneg Note that there will be a KB article change when this is done. Stub a ticket for it.

@xiongjaneg xiongjaneg removed the Needs refining Issue status label Jan 24, 2024
@davidmpickett
Copy link
Contributor

FYI - I updated the Content model to reflect both the possibility of a Front End-only fix, and a more robust Drupal fix

@maxx1128
Copy link

End of Sprint 103 update: I started on this near of the end of the sprint and began a Slack thread to get design/UX feedback on it. It's currently paused to focus on #15959 and is scheduled to discuss at the next UX Sync.

@maxx1128
Copy link

maxx1128 commented Feb 15, 2024

As per the UX Sync discussion on 2/15, I've made the following updates to the pull request:

  • The current pull request associated with this ticket kept the original first two commits, which focus on showing the general mental health phone number instead of the general facility number when needed.
  • The fixes in the last two commits (duplicate number and duplicate appointment headers) were split off and saved into these respective pull requests. As of this writing, they are not associated with any tickets and can be renamed for future stubbed tickets, focusing on the issues they address.
    • va-16652-SAVED-avoid-duplicate-health-service-phone-numbers
    • va-16652-SAVED-avoid-duplicate-appt-headers
  • The last AC in this ticket (using the aria described label) was added to the original pull request since it's one of the original ACs and is a quick win.

@xiongjaneg
Copy link
Contributor

xiongjaneg commented Feb 15, 2024

#17251 stubbed for the duplicate phone number issue
#17253 stubbed for the duplicate appointment header issue

@aklausmeier
Copy link

@thejordanwood FYA of Drupal feasibilities / quirks that you should be looped in on as the designer for FE and Drupal UI

@maxx1128
Copy link

The acceptance criteria for this ticket have been met, but Eli noticed a bug with how the component rendered the phone number with an extension. This didn't appear before since the main facility number has never had an extension (as far as I'm aware). However, the main mental health numbers are sometimes the main facility number with an extension added. I have a draft pull request open for it now. Once it gets merged, this ticket should be good to close.

@xiongjaneg
Copy link
Contributor

@maxx1128 Should this be in prod now? I'm not seeing it in the Cheyenne page
Screenshot 2024-03-11 at 3 35 01 PM

@maxx1128
Copy link

@maxx1128 Should this be in prod now? I'm not seeing it in the Cheyenne page Screenshot 2024-03-11 at 3 35 01 PM

@xiongjaneg It looks like it's working. The main phone in the "Mental Health" service accordion matches the mental health phone number at the top of the page. The two "contact" numbers below are other numbers added to the CMS, so those need to be edited on that side. Are you referring to something else on the page?

@xiongjaneg
Copy link
Contributor

@maxx1128 For the first AC "The Mental health phone number listed under contact information should also appear as the Mental health phone number" I understood that to be the label in both places should be "Mental health care." That's the label in the facility contact information but not in the mental health accordion, though the numbers look correct.

@mmiddaugh Was that your intent?
Screenshot 2024-03-12 at 9 14 06 AM
Screenshot 2024-03-12 at 9 14 35 AM

@xiongjaneg
Copy link
Contributor

The label is out of scope for this ticket. It is covered in #17373. This is good to go.

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 UX VA.gov frontend CMS team practice area VAMC CMS managed product owned by Facilities team
Projects
None yet
Development

No branches or pull requests

6 participants