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
282 stars 203 forks source link

Updates to 10-10d notification emails #88164

Closed baltes closed 2 months ago

baltes commented 3 months ago

As a CHAMPVA applicant, I need info about my files that were received by VA and other helpful guidance in my email notification.

Acceptance criteria

This is a task to update variables for the 10-10d VA-notify template.

Below is an example from form 10-10EZ that we can borrow from.

image.png

Here's what we currently have in our template:

image.png

Change the metadata key for ((updated_at)) to ((date_submitted)) so that it's clear that it's the date the form is submitted rather than a different update date.

Add a variable with a count for all supporting documents update that does not include the form itself. The user should see something to the effect of "We've received your form 10-10d and the X supporting documents you provided." We'll need a variable we can use in VA-notify to populate X.

Add a form name variable for use in VA-notify along with the existing for number. Use names as shown below:

Form number Form name
10-10d Application for CHAMPVA Benefits
10-7959a CHAMPVA Claim Form
10-7959c Other Health Insurance (OHI) Certification
10-7959f-1 Foreign Medical Program (FMP) Registration Form
10-7959f-2 Foreign Medical Program (FMP) Claim Cover Sheet
Add variable for call center phone number so that we can add a key for either the CHAMPVA number or the FMP number. program call center number
Foreign Medical Program (FMP) 877-345-8179
Civilian Health and Medical Program of the Department of Veterans Affairs (CHAMPVA) 800-733-8387
balexandr commented 2 months ago

First PR is for updates to the data model https://github.com/department-of-veterans-affairs/vets-api/pull/17651

balexandr commented 2 months ago

Second PR was merged https://github.com/department-of-veterans-affairs/vets-api/pull/17655

balexandr commented 2 months ago

Third PR with email fix and phone_number https://github.com/department-of-veterans-affairs/vets-api/pull/17667

balexandr commented 2 months ago

@baltes form_name, file_count and phone_number have been added to the email template

Screenshot 2024-07-23 at 4.10.34 PM.png
baltes commented 2 months ago

@balexandr Can you double check the file count and confirm that it is only for supporting docs, not including the form itself?

This submit was for one applicant, with one file for marriage license, two files for insurance cards, and one file for OHI.

image.png
balexandr commented 2 months ago

@baltes It includes the form itself as it is still considered a file that's being processed. I can change it to pass "0" along to the template so it says 0 supporting documents received. I don't see a way to conditionally hide the "File Count" bit in the template if it's 0 FYI.

baltes commented 2 months ago

@balexandr It was always meant to be a count of supporting documents that doesn't include the form itself. No need to hide it if the count is zero.

baltes commented 2 months ago

@balexandr We still need the count changed so that it covers supporting docs and doesn't include the form. Let me know if we need to break that into a separate ticket.

balexandr commented 2 months ago

@baltes So right now it's doing the count based on the form_uuid so if there are multiple applicants with multiple PDFs for the form it will be all those minus the main one. Is that good? I'm basically putting in logic to take the entire count and subtract 1

baltes commented 2 months ago

@balexandr The goal is to tell the user "we received your application and X supporting documents." The user shouldn't have to be aware that we generated multiple form PDFs--it's just one application as far as they're concerned.

I'll close this ticket and break that part out into a new one if that makes sense at this point.

balexandr commented 2 months ago

That's okay I think I got it

balexandr commented 2 months ago

Another PR in review for this change https://github.com/department-of-veterans-affairs/vets-api/pull/17770

balexandr commented 2 months ago
Screenshot 2024-07-30 at 1.16.25 PM.png

@baltes I did a submission with a birth certificate and front and back of medicare cards