-
Notifications
You must be signed in to change notification settings - Fork 69
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
[10-10EZ] Add download_pdf endpoint for 10-10EZ #20968
base: master
Are you sure you want to change the base?
Conversation
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.
LGTM! Do we want to add this new route to the swagger docs?
|
78228f3
to
18d83bf
Compare
18d83bf
to
04ca1a5
Compare
04ca1a5
to
24dc1cf
Compare
24dc1cf
to
a38e6e0
Compare
a38e6e0
to
951b375
Compare
951b375
to
de8c033
Compare
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 like there's some PDF errors in the tests. Rerunning to see if anything changes.
Just updated the branch to see if that helps the failing test. |
f1552c8
to
cc4b97c
Compare
Summary
download_pdf
endpoint in thehealth_care_applications_controller
that receives a post body of form data and generates and fills out a 10-10EZ form pdf that is returned in the response.Related issue(s)
Testing done
Screenshots
Note: Optional
What areas of the site does it impact?
(Describe what parts of the site are impacted andifcode touched other areas)
Acceptance criteria