Skip to content

EROPSPT-717: Remove unused columns from postal and proxy tables#211

Open
aleyipsoftwire wants to merge 2 commits into
remove-retention-statusfrom
remove-retention-status-2
Open

EROPSPT-717: Remove unused columns from postal and proxy tables#211
aleyipsoftwire wants to merge 2 commits into
remove-retention-statusfrom
remove-retention-status-2

Conversation

@aleyipsoftwire

Copy link
Copy Markdown
Contributor

Describe your changes

This PR is stacked onto #210. That PR should be merged first.

Issue ticket number and link

https://mhclgdigital.atlassian.net/browse/EROPSPT-717

Checklist before requesting a review

Tick all those which are done
Replace check box with ❌ if the step is not relevant for this PR

  • I double checked that ACs on the ticket are met by this code update
  • I have checked any risky steps with my TL (cheat sheet for safe release here)
  • ❌ I have updated the relevant yml versions
  • I have added tests to new code and updated existing tests where needed
  • I have checked complies with security policy and data protection policy
  • Code reviewer has verified during code review the relevant controls and practices for the two policies mentioned above
  • I have documented any release dependencies (e.g. service release order or incompatible versions) in the Release notes
    • Can leave tags as tbd until merged

@aleyipsoftwire aleyipsoftwire force-pushed the remove-retention-status-2 branch from bf6df95 to 9143676 Compare June 5, 2026 13:33
@aleyipsoftwire aleyipsoftwire requested review from emisto3 and minalui June 5, 2026 13:34

@emisto3 emisto3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry for all the migration questions! We love DBs 😅

http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-4.1.xsd">

<changeSet author="alex.yip@softwire.com" id="0043_EROPSPT-717_remove_unused_columns_from_postal_and_proxy" context="ddl">
<dropColumn tableName="postal_vote_application" columnName="retention_status"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to worry about locking the DB, what would fail?

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