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

Add eligibility check for contract start date #2943

Merged
merged 2 commits into from
Jul 1, 2024

Conversation

rjlynch
Copy link
Contributor

@rjlynch rjlynch commented Jun 28, 2024

We want to reject claimants if their contract start date is earlier than
6 months before the autumn term. This commit updates the eligibility
checker to do so and also adds some missing tests.

Walk through

contract-start-date.mov

@rjlynch rjlynch added the deploy Deploy a review app for this PR label Jun 28, 2024
rjlynch added 2 commits July 1, 2024 09:53
We want to reject claimants if their contract start date is earlier than
6 months before the autumn term. This commit updates the eligibility
checker to do so and also adds some missing tests.
We eventually want to get rid of the policy models, storing their data
on the claim itself, so the policy eligibility checker is a better home
for this method.
@rjlynch rjlynch force-pushed the CAPT-1747/reject-ineligible-contract-start-dates branch from d97e6e1 to bceb2da Compare July 1, 2024 09:18
@rjlynch rjlynch merged commit 3fd9cbf into master Jul 1, 2024
16 checks passed
@rjlynch rjlynch deleted the CAPT-1747/reject-ineligible-contract-start-dates branch July 1, 2024 09:30
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.

2 participants