LG-13725 Drop aamva column from doc_auth_logs#11052
Conversation
This column was added to the `ignored_columns` list in 85da36e. This commit follows up on that by adding a migration to drop the column. [skip changelog]
aduth
left a comment
aduth
left a comment
There was a problem hiding this comment.
Should we remove this from ignored_columns?
identity-idp/app/models/doc_auth_log.rb
Line 14 in 09720cb
And on a related note, should we go ahead and drop those other columns? I think I asked about this before, and it seemed like they should be removed.
|
Previous discussion: https://gsa-tts.slack.com/archives/C0NGESUN5/p1713188223211769 |
|
The other columns were already dropped. I opened a PR to remove them from the ignored column list here: #11051 I am working on a commit to remove |
|
Oops, just saw your other pull request. |
Is there a reason for it to be separate? I thought it was safe to remove |
The `aamva` column was dropped in #11052. This commit follows-up on that by removing it from the `ignored_columns` list on the model. [skip changelog]
The `aamva` column was dropped in #11052. This commit follows-up on that by removing it from the `ignored_columns` list on the model. [skip changelog]
3 participants
This column was added to the
ignored_columnslist in 85da36e. This commit follows up on that by adding a migration to drop the column.