18F / identity-idp

Login.gov Core App: Identity Provider (IdP)
https://secure.login.gov/
Other
524 stars 112 forks source link

Dmm/add query to protocol report #11339

Closed Sgtpluck closed 3 weeks ago

Sgtpluck commented 3 weeks ago

🎫 Ticket

Link to the relevant ticket: Add facial match issuer query to weekly reports

🛠 Summary of changes

This change

Note: The reorganization was very noisy, so I split this PR into two commits. The first commit shows the reorganization, the second commit is the meat of the actual change.

zachmargolis commented 3 weeks ago

Can we add a screenshot of the report to the PR? (with fake/sample data as needed)? Helps to put the changes in context

lmgeorge commented 3 weeks ago

LGTM! (+1 to all of Zach's comments, but none of them are blocking to me.)