Skip to content

LG-7145 Use validated attributes from AAMVA to cover failed attributes from resolution#7177

Merged
jmhooper merged 6 commits intomainfrom
jmhooper-resolution-result-adjudicator
Oct 19, 2022
Merged

LG-7145 Use validated attributes from AAMVA to cover failed attributes from resolution#7177
jmhooper merged 6 commits intomainfrom
jmhooper-resolution-result-adjudicator

Conversation

@jmhooper
Copy link
Contributor

Previous commits made the State ID Proofer (AAMVA) and the Resolution Proofer (LexisNexis InstantVerify) aware of specific attributes that passed and failed. The purpose of this work was to allow attributes that failed during resolution (namely dob and address) to be covered by attributes verified by the State ID Proofer.

This commit makes a number of changes to that effect:

  1. A request is sent to the state ID proofer even if the resolution proofer fails
  2. If the attributes that failed the resolution proofer check pass the state ID proofer checks that is considered a pass

A new adjudicator class was created to house this logic.

…s from resolution

Previous commits made the State ID Proofer (AAMVA) and the Resolution Proofer (LexisNexis InstantVerify) aware of specific attributes that passed and failed. The purpose of this work was to allow attributes that failed during resolution (namely dob and address) to be covered by attributes verified by the State ID Proofer.

This commit makes a number of changes to that effect:

1. A request is sent to the state ID proofer even if the resolution proofer fails
2. If the attributes that failed the resolution proofer check pass the state ID proofer checks that is considered a pass

A new adjudicator class was created to house this logic.

[skip changelog]
@jmhooper jmhooper requested a review from kbighorse October 19, 2022 19:11
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

love this!

@jmhooper jmhooper merged commit 15644ac into main Oct 19, 2022
@jmhooper jmhooper deleted the jmhooper-resolution-result-adjudicator branch October 19, 2022 20:45
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.

2 participants