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
Validate dob #186989077 #4463
Validate dob #186989077 #4463
Conversation
…r-warning-#186829970
Heroku app: https://gyr-review-app-4463-ac44ff4d2273.herokuapp.com/ |
</div> | ||
<% elsif ff.object.ctc_qualifying %> | ||
<div class="form-group"> | ||
<%= ff.warning_for_date("#{ff.object_name}[dob_year]", MultiTenantService.statefile.end_of_current_tax_year.years_ago(17), nil, nil, t(".dependent_dob_under_17_warning")) %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider extracting MultiTenantService.statefile.end_of_current_tax_year
so that here and @sixty_five_years_ago
can both use it and not be inconsistent about extracting a variable for one age and not the other.
closing for now. will bring back if still relevant for FYST 2025 |
Warning for cases where return indicates that the user was over 65 but the DOB was under 65:
Warning for cases where return indicates that the user was under 65 but the DOB was over 65: