-
Notifications
You must be signed in to change notification settings - Fork 14
Nj 366 dep health ins #6071
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
Nj 366 dep health ins #6071
Conversation
|
Heroku app: https://gyr-review-app-6071-b6aa60566c7f.herokuapp.com/ |
| it "does not show" do | ||
| it "shows" do | ||
| allow_any_instance_of(StateFileNjIntake).to receive(:has_health_insurance_requirement_exception?).and_return(false) | ||
| allow_any_instance_of(StateFileNjIntake).to receive(:eligibility_all_members_health_insurance_no?).and_return(false) |
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.
[dust] - I think these two lines could optionally be removed now like they are for the no dependents scenario above, right?
allow_any_instance_of(StateFileNjIntake).to receive(:has_health_insurance_requirement_exception?).and_return(false)
allow_any_instance_of(StateFileNjIntake).to receive(:eligibility_all_members_health_insurance_no?).and_return(false)
|
I don't understand how the question could have been incorrect before -- it's still 2025 so how can we ask people about this year when tax filing for 2025 hasn't started yet? Were we supposed to ask back in January 2025 about the first few weeks of the year already? Put another way, I've never seen present tense in a tax filing question so that confuses me. Is it because getting an extension this late into the year actually changes the rules and people are now asked about this year as well? All that being said, I'm presuming the answer is just that I don't know this corner of the tax filing world and all is well... and given all that, from a technical standpoint I'm a 👍 |
|
@mluedke2 Apparently the only reason taxation includes this question is to share this data with an outreach team. They care about whether or not your dependents currently have health insurance so they can get enrolled. It does not affect the tax calculation. |
|
Ahhhh ok. That makes way more sense. Thanks! |
|
Just tested, and this looks good to me! |
Change logic to show NJ health insurance question to anyone with dependents, update copy to current year
Link to pivotal/JIRA issue
https://github.com/newjersey/affordability-pm/issues/366
Is PM acceptance required? (delete one)
What was done?
The dependents health insurance question was not correct before. It should instead concern the current year and be shown to anyone with dependents.
How to test?
Screenshots (for visual changes)
Copy was updated
