-
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 mapping for right side page 2 13614c #5446
Add mapping for right side page 2 13614c #5446
Conversation
Heroku app: https://gyr-review-app-5446-4c781239522c.herokuapp.com/ |
"form1[0].page1[0].spousesLastName[0]" => 'Heifer', | ||
"form1[0].page1[0].spousesTelephoneNumber[0]" => nil, | ||
"form1[0].page1[0].spousesDateOfBirth[0]" => "11/1/1959", | ||
"form1[0].page1[0].spousesJobTitle[0]" => nil, |
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.
removed two lines that were duplicates
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.
spacing changes were just a result of command + option + L
that reformatted the file
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.
no need to change here, but it’s better to avoid mixing in unrelated cosmetic changes since it clutters the PR
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
…-2-make-sure-collected-fields-in-the-edtiable-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)
updated pdf:
when hub fields look like:
![Screenshot 2025-01-24 at 7 17 57 PM](https://private-user-images.githubusercontent.com/49880002/407036942-c096f501-61bc-448a-9153-f61dfbf90c3f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkyNzcyMjgsIm5iZiI6MTczOTI3NjkyOCwicGF0aCI6Ii80OTg4MDAwMi80MDcwMzY5NDItYzA5NmY1MDEtNjFiYy00NDhhLTkxNTMtZjYxZGZiZjkwYzNmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjExVDEyMjg0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPThkZDI4ZGE0ODA5NmJjOWE2ODVkOTlmZDhlOGU1NTc1Yjc3MGU3YzliOTM3ZTMxOGVlODVjZWQ0NDAzMzIxZjUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.RmrgS20T8lJxivavHILv-SAcQ9qTwGvVgV5_lD_TvQM)
![Screenshot 2025-01-24 at 7 17 45 PM](https://private-user-images.githubusercontent.com/49880002/407036960-dea50979-8674-4367-bc8a-f90295429908.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkyNzcyMjgsIm5iZiI6MTczOTI3NjkyOCwicGF0aCI6Ii80OTg4MDAwMi80MDcwMzY5NjAtZGVhNTA5NzktODY3NC00MzY3LWJjOGEtZjkwMjk1NDI5OTA4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjExVDEyMjg0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWUzYmY5ZmUwN2E2YzhmZmUzNGQ2ZDdiNmQ2NTM1MzI0YTY0YjNiZjBjMDRjNjJiNTU2NDliNTg5ZWZlZWFkNjMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.JOcZd1PfvthszD1OkSR8TOzS-Nn5GoSwrJn_iKee0qo)