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
281 stars 202 forks source link

Spool File Routing: Files being routed to St. Louis on 08/24 #12818

Closed desireeturner closed 4 years ago

desireeturner commented 4 years ago

What happened?

Shay Norton-Leonard from the Buffalo EDU team sent @WillMcCormack the following information regarding spool file routing for routing that took place on 08/24/20: 12:08 PM] Norton-Leonard, Shay, VBAVACO Will - I also was looking to see if the spool file has been processed and see that yesterday 8/24 there was a spool file created for RPO 331 (St. Louis). There has no been one for a while... since July. [12:09 PM] Norton-Leonard, Shay, VBAVACO The TIMS team is running all 331 files in 307, but I found it a little odd that one was created at all. Not sure if it is related to testing or not. [12:18 PM] Norton-Leonard, Shay, VBAVACO And... it did not just have one or 2 claims, but 303.

Shay also checked the file and did not see any duplicate submissions

After some investigation, @shawkey-daniel found that this happened: For the spool file... We had moved all of the central states to eastern. Somehow those ended up back in central. The routing should still work, because we removed central from a possibility, but I think we need a defect to move those central states to the eastern array.

Steps to Reproduce

Desired behavior

Logic for routing is in https://github.com/department-of-veterans-affairs/vets-api/blob/master/app/workers/education_form/education_facility.rb

Only Buffalo and Muskogee should be getting spool files

Acceptance Criteria

How to configure this issue

devin-mccurdy commented 4 years ago

PR: https://github.com/department-of-veterans-affairs/vets-api/pull/4786

delli-gatti-michael commented 4 years ago

Submitting using a user in Colorado (previously a Central state) routes to Buffalo

Screen Shot 2020-08-27 at 11.29.19 AM.png

delli-gatti-michael commented 4 years ago

Confirmed with EDU that this is no longer an issue. Closing this defect.