Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API-8244: SpecialIssueUpdater Background Job Error #7332

Merged
merged 3 commits into from
Jul 6, 2021

Conversation

SlakrHakr
Copy link
Contributor

Description of change

Unexpected error was noticed in sentry with regards to our background job to update special issues for a claim submission.
This appears to be happening in the situation where the Veteran has more than one contention but none of them match the particular claims that was submitted.

Original issue(s)

https://vajira.max.gov/browse/API-8244

Things to know about this PR

No new tests are present because there was already an existing test for this exact situation. It was unfortunately already "passing" because it simply detected if an exception was raised (which causes the job to be queued back up).

@SlakrHakr SlakrHakr added the Lighthouse lighthouse label Jun 30, 2021
@SlakrHakr SlakrHakr self-assigned this Jun 30, 2021
@SlakrHakr SlakrHakr requested review from a team as code owners June 30, 2021 21:53
@SlakrHakr SlakrHakr requested a review from ericboehs June 30, 2021 21:53
@va-vfs-bot va-vfs-bot temporarily deployed to slakrhakr/API-8244/master/master June 30, 2021 22:04 Inactive
jw81
jw81 previously approved these changes Jul 1, 2021
hinzed1127
hinzed1127 previously approved these changes Jul 1, 2021
@SlakrHakr SlakrHakr dismissed stale reviews from hinzed1127 and jw81 via d848f41 July 1, 2021 18:51
@SlakrHakr SlakrHakr force-pushed the slakrhakr/API-8244 branch 2 times, most recently from d848f41 to 2e3c299 Compare July 2, 2021 15:06
hinzed1127
hinzed1127 previously approved these changes Jul 2, 2021
hinzed1127
hinzed1127 previously approved these changes Jul 2, 2021
@va-vfs-bot va-vfs-bot temporarily deployed to slakrhakr/API-8244/master/master July 2, 2021 18:35 Inactive
@SlakrHakr SlakrHakr merged commit a951ae8 into master Jul 6, 2021
@SlakrHakr SlakrHakr deleted the slakrhakr/API-8244 branch July 6, 2021 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Lighthouse lighthouse
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants