-
Notifications
You must be signed in to change notification settings - Fork 70
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
Comments
Questions to be asked:
Front end:
|
@xiongjaneg This is a front end change that would need change management. |
@davidmpickett Please review for any related impact to/from #16655 |
FE change needed = look at service, if it's mental health, use the number specified by the facility |
@xiongjaneg Note that there will be a KB article change when this is done. Stub a ticket for it. |
FYI - I updated the Content model to reflect both the possibility of a Front End-only fix, and a more robust Drupal fix |
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. |
As per the UX Sync discussion on 2/15, I've made the following updates to the pull request:
|
@thejordanwood FYA of Drupal feasibilities / quirks that you should be looped in on as the designer for FE and Drupal UI |
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. |
@maxx1128 Should this be in prod now? I'm not seeing it in the Cheyenne page |
@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? |
@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? |
The label is out of scope for this ticket. It is covered in #17373. This is good to go. |
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.
Problems to solve
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
Within the Mental health service accordion
Cheyenne VA Medical Center
Phone numbers displayed in Location and contact information area
Within the Mental health service accordion
Engineering notes
Acceptance Criteria
The text was updated successfully, but these errors were encountered: