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

Fix claims being flagged for qa #3252

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

rjlynch
Copy link
Contributor

@rjlynch rjlynch commented Oct 1, 2024

When a new academic year has started, and we are yet to approved any claims from this academic year, all claims from the previous year are flagged for qa when approved. To avoid this we need to count the number of approved claims in the claim's academic year rather than the current academic year.

@cwadsworthdfe
Copy link

Thanks for jumping on this @rjlynch

@rjlynch rjlynch added the deploy Deploy a review app for this PR label Oct 2, 2024
When a new academic year has started, and we are yet to approved any
claims from this academic year, all claims from the previous year are
flagged for qa when approved. To avoid this we need to count the number
of approved claims in the _claim's_ academic year rather than the
current academic year.
@rjlynch rjlynch force-pushed the CAPT-1835/all-claims-approved-flagged-for-qa branch from b0ff872 to 2fe90f5 Compare October 2, 2024 10:38
@rjlynch rjlynch merged commit 4c05d5e into master Oct 2, 2024
14 checks passed
@rjlynch rjlynch deleted the CAPT-1835/all-claims-approved-flagged-for-qa branch October 2, 2024 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Deploy a review app for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants