Skip to content

LG-11979 remove aamva banlist config#9909

Merged
soniaconnolly merged 1 commit intomainfrom
sonia-lg-11979-remove-aamva-banlist-config
Jan 11, 2024
Merged

LG-11979 remove aamva banlist config#9909
soniaconnolly merged 1 commit intomainfrom
sonia-lg-11979-remove-aamva-banlist-config

Conversation

@soniaconnolly
Copy link
Contributor

@soniaconnolly soniaconnolly commented Jan 11, 2024

🎫 Ticket

LG-11979

🛠 Summary of changes

Remove aamva_sp_banlist_issuers unused config value.

It was empty in application.yml.default, and it doesn't appear in the s3 buckets for any env.

📜 Testing Plan

  • Run specs. This config was unused.
  • To make really sure, create account, start IdV, continue up to submitting VerifyInfo step.

@soniaconnolly soniaconnolly requested a review from a team January 11, 2024 18:57
It was empty in application.yml.default, and it doesn't appear in the s3 buckets for any env.

[skip changelog]
@soniaconnolly soniaconnolly force-pushed the sonia-lg-11979-remove-aamva-banlist-config branch from 0304cd3 to 539669a Compare January 11, 2024 19:48
Copy link
Contributor

@solipet solipet left a comment

Choose a reason for hiding this comment

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

LGTM!

@soniaconnolly soniaconnolly merged commit 43e0c4a into main Jan 11, 2024
@soniaconnolly soniaconnolly deleted the sonia-lg-11979-remove-aamva-banlist-config branch January 11, 2024 20:22
@soniaconnolly soniaconnolly changed the title LG0-11979 remove aamva banlist config LG-11979 remove aamva banlist config Jan 11, 2024
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