-
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
Add page 1 certified volunteer section dependents 13614c #5453
Add page 1 certified volunteer section dependents 13614c #5453
Conversation
Heroku app: https://gyr-review-app-5453-ba796e9e5486.herokuapp.com/ |
@@ -608,8 +648,8 @@ | |||
expect(intake_pdf.hash_for_pdf[additional_comments_key]).to eq(<<~COMMENT.strip) | |||
Other income types: garden gnoming | |||
Additional Dependents: | |||
(a) Polly Pony (b) 8/27/2018 (c) Baby (d) 5 (e) N (f) Y (g) S (h) N (i) Y CVP: //// | |||
(a) Patrick Pony (b) 3/11/2019 (c) Son (d) 8 (e) N (f) Y (g) S (h) N (i) N CVP: //// | |||
(a) Polly Pony (b) 8/27/2018 (c) Baby (d) 5 (e) S (f) N (g) Y (h) N (i) Y CVP: //// |
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.
i think this will get overwritten in https://github.com/codeforamerica/vita-min/pull/5431/files#diff-55f9b31b5196260702be6515a3483ad05eca50729a5324079b6c49fcd71250df
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.
approved
…-1-make-sure-collected-fields-in-the-editable-13614-c-populates-the-pdf-form-in-the-correct-places
…-1-make-sure-collected-fields-in-the-editable-13614-c-populates-the-pdf-form-in-the-correct-places
Link to pivotal/JIRA issue
Is PM acceptance required? (delete one)
Reminder: merge main into this branch and get green tests before merging to main
What was done?
How to test?
Screenshots (for visual changes)