-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CST - Added db migration to update evidence_submissions fields from type string to integer #20490
CST - Added db migration to update evidence_submissions fields from type string to integer #20490
Conversation
…mission_field_data_types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmclaren19 sorry I didn't catch this sooner, but there needs to be data migration for a column type change
There isn't any data in the table currently, is there a need to worry about data migrations in that case? |
Summary
This pr is related to the work from #20105. As part of that work we need to update the request_id, claim_id and tracked_item_id to be integers instead of strings. Currently the table evidence_submissions is not being used and is behind the feature flag
cst_send_evidence_submission_failure_emails
which is currently disabled in upper environments.This pr is currently blocking the pr here.
Related issue(s)
-Link to ticket created in va.gov-team repo department-of-veterans-affairs/va.gov-team#100037
Testing done
What areas of the site does it impact?
Claim Status Tool
Acceptance criteria
Requested Feedback
(OPTIONAL)What should the reviewers know in addition to the above. Is there anything specific you wish the reviewer to assist with. Do you have any concerns with this PR, why?