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

SPIKE: Where do we cut VBA names? What can we do without lighthouse? #17351

Closed davidmpickett closed 3 days ago

davidmpickett commented 6 months ago

User Story or Problem Statement

Follow up from #15752 (VBA IA)

What can we do without lighthouse?

VBA Naming Logic_2024-02-27_17-29-49

VBA Naming Logic_2024-02-27_17-30-01

VBA Naming Logic_2024-02-27_17-29-56

Once lighthouse is ready to accept new fields

VBA Naming Logic_2024-02-27_17-32-39

Acceptance Criteria

davidmpickett commented 5 months ago

Copy and pasting this comment https://github.com/department-of-veterans-affairs/va.gov-cms/issues/15752#issuecomment-1959742070

For the engineering conversation, there are four locations which have shared vha locations which are not reflected in the name

Current name vha shared location id vha shared location name
White River Junction VA Regional Benefit Office vha_405 White River Junction VA Medical Center
Cheyenne VA Regional Benefit Office vha_442 Cheyenne VA Medical Center
Muskogee VA Regional Benefit Office vha_623 Jack C. Montgomery Department of Veterans Affairs Medical Center
San Diego VA Regional Benefit Office vha_664BY Kearny Mesa VA Clinic
and a few examples with non-VA shared locations which are not reflected in the name (one is an MVP location) Current name non-VA shared location name
Cleveland VA Regional Benefit Office Patrick V. McNamara Federal Building
Detroit VA Regional Benefit Office Anthony J Celebrezze Federal Building
St Paul VA Regional Benefit Office Henry Whipple Federal Building
Albuquerque VA Regional Benefit Office Dennis Chavez Federal Building

The rule about having either a shared non-VA location or a shared VA location but not both is not accurate: Manila VA Regional Benefit Office at U.S. Embassy in Manila is associated with a non-VA building (U.S. Embassy in Manila) as well as a shared VHA location (VA Manila Outpatient Clinic). Both are true: the Clinic is on the Embassy campus.

Questions

jilladams commented 1 month ago

I originally pulled this ticket into Next Refinement because it's a part of the VBA National epic and because we talked about #15583 in relation to it this week.

However: I'm not actually 100% certain on priority relative to other Q3 / VBA things. It might be best to let @Agile6MSkinner work through the VBA National epic priorities with Michelle before we try to dive in here, since that will be happening imminently.

Moving back to Backlog.

jilladams commented 3 weeks ago

This is a Drupal-first question, and then we can figure out how FE works.

omahane commented 2 weeks ago

Of the 489 VBA sites in prod, we have many that are only distinguishable by a combination of

These include the facilities that begin with the following:

@mmiddaugh (cc: @davidmpickett) Am I right in assuming that these are not up for the same kind of copy and bifurcation of the name that we are spiking in this ticket?

If that's the case, it's looks we are only looking to change the name of a dozen or so:

Am I reading this right? Please correct me of I'm mistaken.

davidmpickett commented 2 weeks ago

@omahane, because only Regional Offices and Satellite offices are in scope for MVP, my previous work focused only on those two types. We aren’t solving this for all VBA facility types.

This spreadsheet was my working doc for tracking any facilities that might be edge cases https://dvagov.sharepoint.com/:x:/s/SitewideFacilitiesTeam/ESGDTL0pkqdDiKT3oqcFoZEBuEtSx0NdGZIbvUwY7TwQ9w?e=9amPaQ

davidmpickett commented 2 weeks ago

Note also that for Satellite Offices that don’t currently have a geographical region prefixed before the office type, we have a ticket for @mmiddaugh to discuss those with stakeholders so that we then could cut off the shared location

https://github.com/department-of-veterans-affairs/va.gov-cms/issues/17358

omahane commented 2 weeks ago

@omahane, because only Regional Offices and Satellite offices are in scope for MVP, my previous work focused only on those two types. We aren’t solving this for all VBA facility types.

I was looking at all VBA facility names and found that any solution to the problem identified in this ticket appears to apply to dozens (among the hundreds). Can you confirm that we are only concerned right now with the list I posted above? I am trying to both get an understanding of the problem set and conceive of viable solutions for that set.

omahane commented 1 week ago

After talking looking at the naming conventions and talking to @davidmpickett, I have the following suggestions:

VBA Facility content type and migration changes

VBA facility workflow changes

Other changes to the workflow could include:

Lighthouse changes (longer term)

omahane commented 1 week ago

@davidmpickett Do you have any additional thoughts?

davidmpickett commented 1 week ago

Legend of different names for names

Machine name Nickname Meaning Notes
name API Name What Lighthouse API lists as the name of a given facility Source of truth is actually Sandy's DB
title (now ) Drupal name The title of a node in Drupal drives URLs, breadcrumbs etcetera Right now this is populated with API Name
official name Official name in Drupal This will be a new field that will match the API name Won't actually do much, aside from hold the API name in case we need it for something
title (future) Common name / Condensed name / Drupal name After creating official name, we will make it so this Drupal property can be overridden to shorten names in Drupal only Follow Vet Center pattern
davidmpickett commented 1 week ago

@davidmpickett Do you have any additional thoughts?

I added a legend of the different terms you are referencing in your proposal to hopefully make it a little easier for others to understand. Otherwise my only thought is "good work" :)

omahane commented 1 week ago

Oh, I noticed that we don't have an alias pattern for VBA Facility Services, but I think we should also follow the Vet Center approach here, too. I made the ticket.

jilladams commented 1 week ago

@mmiddaugh @Agile6MSkinner ready for final signoff from Product.

omahane commented 3 days ago

Product decided during the 12 Sept 2024 Sitewide UX sync to proceed with the recommendations I made in the above comment. I am writing up a ticket to capture the recommendation. This ticket is now closable.