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

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
97 stars 69 forks source link

Widgets: Replace facility name with system name #9074

Closed wesrowe closed 2 years ago

wesrowe commented 2 years ago

Timing note

Let's get Drupal SSOT out into prod and bug-free before attempting this change.

Description

Prereq: #9787 https://www.va.gov/data/cms/vamc-ehr.json -- provides Facility Name / System name. In this file, all Facilities are associated with a System.

As a Veteran I want to be shown health system names instead of facility names, so that I can more easily ascertain which button to click to complete my EHRM-related task.

This issue is only to change the text that is displayed per system – e.g., replace "Wylie P Chalmers" with "VA Central Ohio".

Current state for a multi-Cerner patient:

image

Assumptions:

Tasks

Acceptance Criteria

CMS Team

Please check the team(s) that will do this work.

jilladams commented 2 years ago

If someone picks up this ticket, confer with @ryguyk for clarity about current state of Drupal SSOT.

jilladams commented 2 years ago

Note: there's a bug in Staging, that will be fixed when this PR merges: https://github.com/department-of-veterans-affairs/vets-website/pull/21809

jilladams commented 2 years ago

@ryguyk @allisonlu did you all decide to leave this ticket with Allison or that Ryan you would handle both Facility name issues?

ryguyk commented 2 years ago

@jilladams I'm going to take this. I've already begun work on it. It's in good shape overall with just a few question marks that are not blockers or major cause for concern.

ryguyk commented 2 years ago

PR pushed for this, but awaiting external review from Release Tools. We've been waiting over 24 hours already, so I'm hopeful this should be approved Monday. It should merge Monday as well, provided that review comes before deployment cutoff. FWIW, it's a change I certainly don't want to merge on a Friday anyway, as it involved a lot of moving things around.

ryguyk commented 2 years ago

PR approved and merged. Closing issue.