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

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
282 stars 203 forks source link

[FE, QA] 10-10EZ - Facility Selection: QA Facilities API functionality #72257

Closed hdjustice closed 8 months ago

hdjustice commented 10 months ago

Background

The static list used today is very outdated and causing issues for Veterans who are completing the 10-10EZ form. We have connected to the Facilities API in lower level environments with success, and need to implement this functionality in Production. This will allow a dynamic list of facilities to populate a dropdown menu, which will always be a current list and remove the manual maintenance from the 10-10 team.

This is an interim fix of the static list issue. We have plans to redesign the facilities selection page to include a more dynamic search functionality, also built for the 10-10cg form.

Tasks

Acceptance Criteria

fbonds commented 9 months ago

Spot checking second task: comparing facility results to Facilities Search Page for random selection of states (5)

fbonds commented 9 months ago

Submitted issue to Lighthouse regard discrepancy between what we're displaying and available VA clinics in Kansas (yesterday)

"Hi our Application built for va.gov accesses the following uri to get a list of VA medical facilities in Kansas.

/v1/facilities/va?type=health&per_page=100&state=KS

Which returns: Colmery-O'Neil Veterans' Adminstration Medical Center Dwight D. Eisenhower Department of Veterans Affairs Medical Center Robert J. Dole Department of Veterans Affairs Medical and Regional Office Center Paola VA Clinic Fort Riley CBOC Shawnee VA Clinic

but omits all these other open VA clinics in Kansas: Dodge City VA Clinic Fort Scott VA Clinic Hutchinson VA Clinic Junction City VA Clinic Wyandotte CBOC Lawrence VA Clinic Parsons VA Clinic Salina VA Clinic Overland Park Radiation Oncology VA Clinic

Could you help us to understand why this is happening?"

Awaiting their reply

emily-brown34 commented 9 months ago

Still in conversations with Lighthouse regarding the above as of 1/17

fbonds commented 9 months ago

The mismatch in results for Kansas seem to be a mismatch in product, not in stage. Our update in Stage actually is reporting correct result set. I am unable to reproduce my original incorrect result set and subsequently have determined somehow I accidently was testing prod. Marking this step as complete.

fbonds commented 9 months ago

The next task for QA on this ticket (re: Check submission with JF) seems to be blocked by issue arising in Slack thread

fbonds commented 8 months ago

Verified Facility selection through application submittal with Joshua Faulkner in stage for the following users: Neil Armstrong Spokane VA Mobile Clinic - Washington Michael Collins Toledo VA Clinic - Ohio Buzz Aldrin, Jr Southeast Washington VA Clinic - DC

QA tasks complete

hdjustice commented 8 months ago

Great, thank you @fbonds !!

hdjustice commented 8 months ago

Chapley to add a comment here with next steps, Heather to create ticket

jadon1979 commented 8 months ago

The issue here is that Validations for EZ are handled through vets-json-schema. The validation checks preferred facility against an enum (list) of facilities in the schema. Lighthouse supplies more facilities than we have in the enum so any selected facility from lighthouse, that is not in the schema, will trigger a validation failure.

We need to update vets-json-schema to look explicitely for a string rather than an enum.
Afterwards we will update vets-website with the new vets-json-schema version number and remove the facilities API feature flag from the code base.
Vets-json-schema will need to be deployed. followed closely by vets-website.

Once these are done then we can update vets-api to remove hca_use_facilities_API from the config/features.yml file