department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
283 stars 204 forks source link

[MVP] Figure out what creates the PDFs for Pension form #68913

Closed mjknight50 closed 11 months ago

mjknight50 commented 1 year ago

Issue Description

When central mail or lighthouse benefits intake API are used, a PDF is created in the process.

https://app.mural.co/t/departmentofveteransaffairs9999/m/departmentofveteransaffairs9999/1697740873991/4be42fcadc013be6b3d65bd620522ee68e8e0faf?wid=0-1698863820992

For additional context and help, see this: https://github.com/department-of-veterans-affairs/vets-website/tree/main/src/applications/pensions/21p527ez#async-and-error-ha[…]for-form-submission

(scroll to "Creates a PDF")

Acceptance Criteria

aplatt-coforma commented 1 year ago

Hey team! Please add your planning poker estimate with Zenhub @mjknight50 @TaiWilkin @ToddWebDev

danlim715 commented 1 year ago

Here's a summary of my investigation into the PDF generation of the Pension form and the scope of work that needs to be done in order to accommodate the new version of the form:

Investigate what creates the PDF when an online Pension form is submitted

Investigate whether the PDF generation service lives in vets-api and doesn't rely on BGS

What is the validation?

On form submission, only 3 validations occur on the SavedClaim model:

What is the process to change the validation and to send new fields?

What is the downtime?

There should be no downtime since all the changes we need to make live in vets-api and vets-json-schema, but this rework could potentially increase the scope of what we originally thought needed to be done.

If you have any questions, please don't hesitate to ask. I'd also be more than happy to meet via Zoom or Slack to summarize my findings and answer any questions in person if necessary.

I've also attached the old and new versions of the PDF here as a reference:

21P-527EZ (old).pdf

VBA-21P-527EZ-ARE (new).pdf

danlim715 commented 1 year ago

@emilytheis @va-albers Please let us know your thoughts on this, or if there's something I missed/misunderstood.