Skip to content

Commit

Permalink
Hide PII data on confirmation pages (#34830)
Browse files Browse the repository at this point in the history
  • Loading branch information
ToddWebDev authored Feb 21, 2025
1 parent a191c3a commit 50e4d38
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 12 deletions.
10 changes: 8 additions & 2 deletions src/applications/burials-ez/containers/ConfirmationPage.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,10 @@ const ConfirmationPage = ({ form, isLoggedIn, route }) => {
<ul className="claim-list">
<li>
<h3>Who submitted this form</h3>
<span>
<span
className="dd-privacy-hidden"
data-dd-action-name="claimant full name"
>
{claimantName?.first} {claimantName?.middle} {claimantName?.last}{' '}
{claimantName?.suffix}
</span>
Expand All @@ -112,7 +115,10 @@ const ConfirmationPage = ({ form, isLoggedIn, route }) => {
</li>
<li>
<h3>Deceased Veteran</h3>
<span>
<span
className="dd-privacy-hidden"
data-dd-action-name="deceased veteran full name"
>
{veteranName.first} {veteranName.middle} {veteranName.last}{' '}
{veteranName.suffix}
</span>
Expand Down
24 changes: 14 additions & 10 deletions src/applications/pensions/containers/ConfirmationPage.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,9 @@ const ConfirmationPage = ({ route }) => {
Your information for this application
</h3>
<h4>Your name</h4>
<p>{fullName}</p>
<p className="name dd-privacy-hidden" data-dd-action-name="full name">
{fullName}
</p>
<h4>Date you submitted your application</h4>
<p>{submittedAt}</p>
{confirmationNumber && (
Expand Down Expand Up @@ -279,23 +281,25 @@ const ConfirmationPage = ({ route }) => {
</p>
</va-alert>
<h4>Bank account you provided in your VA pension application</h4>
<p className="va-address-block">
<p className="va-address-block dd-privacy-hidden">
{data.bankAccount.bankName && (
<>
<span data-dd-action-name="bank name">
{data.bankAccount.bankName}
<br />
</>
</span>
)}
{data.bankAccount.accountNumber ? (
<>
<span data-dd-action-name="bank account number">
{obfuscateAccountNumber(data.bankAccount.accountNumber)}
<br />
</>
</span>
) : null}
{data.bankAccount.accountType ? (
<span data-dd-action-name="bank account type">
{`${data.bankAccount.accountType.charAt(0).toUpperCase() +
data.bankAccount.accountType.slice(1)} Account`}
</span>
) : null}
{data.bankAccount.accountType
? `${data.bankAccount.accountType.charAt(0).toUpperCase() +
data.bankAccount.accountType.slice(1)} Account`
: null}
</p>
<p>
If you currently receive VA benefit payments through direct
Expand Down

0 comments on commit 50e4d38

Please sign in to comment.