-
Notifications
You must be signed in to change notification settings - Fork 13
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 79 - add page to select income source for 1099-R retirement income #5529
base: main
Are you sure you want to change the base?
Conversation
Heroku app: https://gyr-review-app-5529-bc2100d11010.herokuapp.com/ |
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.
Looks good!
app/views/state_file/questions/nj_retirement_income_source/edit.html.erb
Outdated
Show resolved
Hide resolved
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.
Cool to learn about Flipper! One question about the future of the state_specific_followup code.
spec/controllers/state_file/questions/nj_retirement_income_source_controller_spec.rb
Show resolved
Hide resolved
app/controllers/state_file/questions/nj_retirement_income_source_controller.rb
Show resolved
Hide resolved
15f5e04
to
86b24ec
Compare
Link to pivotal/JIRA issue
https://github.com/newjersey/affordability-pm/issues/79
Is PM acceptance required? (delete one)
What was done?
How to test?
Flipper.enable(:show_retirement_ui)
in rails console to enable the UI toggle/en/hub/flipper/features
and enableshow_retirement_ui
Screenshots (for visual changes)