Skip to content

Prevent AAMVA from overriding an address failure when the address is … - #13343

Open
Mawar2 wants to merge 1 commit into
mainfrom
mwarren-LG-17686-address-override
Open

Prevent AAMVA from overriding an address failure when the address is …#13343
Mawar2 wants to merge 1 commit into
mainfrom
mwarren-LG-17686-address-override

Conversation

@Mawar2

@Mawar2 Mawar2 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

changelog: Bug Fixes, Identity Verification, AAMVA no longer overrides an address failure when the address is edited

🎫 Ticket

Link to the relevant ticket:
LG-17686

🛠 Summary of changes

AAMVA verifies the address printed on the ID. If the user edits their residential address after doc auth, resolution proofs an address AAMVA never saw, but a verified AAMVA address could still cover the InstantVerify address failure.

Idv::VerifyInfoController#pii now passes address_edited through to the adjudicator, and ResultAdjudicator drops :address from the AAMVA-verified attributes when it is set. Other verified attributes (dob, ssn, etc.) still cover their corresponding resolution failures.

Remote flow only.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Complete doc auth so AAMVA verifies the address, edit the residential address on the address step, then submit Verify your info. Resolution fails and is not covered by AAMVA.
  • Same flow without editing the address. AAMVA still covers the InstantVerify address failure and proofing passes.
  • Edit the address where AAMVA verified a non-address attribute (for example dob) and InstantVerify fails on that attribute. Coverage still applies.
  • Confirm address_edited appears in the Idv::Agent applicant PII on submit.

…edited

changelog: Bug Fixes, Identity Verification, AAMVA no longer overrides an address failure when the address is edited
@Mawar2 Mawar2 self-assigned this Aug 6, 2026
@Mawar2
Mawar2 requested review from a team and amirbey and removed request for a team August 6, 2026 20:59

@AShukla-GSA AShukla-GSA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shanechesnutt-ft shanechesnutt-ft left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hold on merging this I want Chris to a take a look first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants