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
96 stars 69 forks source link

Front end templates for Vet Center shows both common and official names (for other nearby vet centers) #16309

Open maxx1128 opened 9 months ago

maxx1128 commented 9 months ago

[2024-09-04] - still blocked by #15445 and Lighthouse, because Nearby locations for Vet Centers is driven by Lighthouse (not editor input)

Problem statement

Veterans may know and search for their closest Vet Center by a common name, rather than an official name. Vet Center common names should be displayed along with official names to help Veterans identify and locate their Vet Centers.

This ticket is related to #15424, since updating the names for nearby vet centers on location pages involves changes made in vets-website and vets-api, which is out of the scope of the rest of the ticket work. So changes for that specific part of the page have been moved to this ticket so the rest is not blocked.

Background

The Outstation associated with Salt Lake City Vet Center is displayed using only the Official name. The Common name concept needs to be available for Outstations as well.

Example: Major Brent Taylor Vet Center Outstation (node/17527), associated with Salt Lake City Vet Center

CMS and FE screenshots [Private Zenhub Image](https://api.zenhub.com/attachedFiles/eyJfcmFpbHMiOnsibWVzc2FnZSI6IkJBaHBBZzNkIiwiZXhwIjpudWxsLCJwdXIiOiJibG9iX2lkIn19--28c78c896ee1f339cda8dec32d062c36e2494f64/image.png) Test cases should come from Tugboat data

Current behavior

15424 lists all the current behavior of this bug. This ticket only lists one example (the second of three total examples) for this ticket's specific scope. The first and third examples are to only be addressed in pull requests connected to the original ticket.

Expected behavior

Tickets

Acceptance Criteria

QA

Team

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

maxx1128 commented 9 months ago

While I'm creating the ticket, I will add an estimate of a 5. It will take changes in two repos: vets-website (which should be relatively simple, just the markup at src/applications/static-pages/facilities/vet-center/NearByVetCenters.jsx) and vets-api (which will be more complicated due to adding data at the endpoint at modules/facilities_api/app/controllers/facilities_api/v1/va_controller.rb).

davidmpickett commented 1 week ago

Are we sure this isn't blocked by #15445? Nearby Vet Centers uses the Facilities API and Common Name isn't available in there. Or will this be some kind of joint React and GraphQL partnership?