Skip to content
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

[AQTS-688] New trs_trn_requests table to replace dqt_trn_requests #2547

Conversation

Hassanmir92
Copy link
Collaborator

@Hassanmir92 Hassanmir92 commented Dec 5, 2024

Ticket: https://dfedigital.atlassian.net/browse/AQTS-688

As part our move away from DQT, we hold a table called dqt_trn_requests. We want to rename this table to be trs_trn_requests.

In this PR, we:

  1. Introduce the new trs_trn_requests table.
  2. Ensure all new requests to TRS populate this new table.
  3. We have a rake task to move all existing dqt_trn_requests to trs_trn_requests

Release order:

  1. Merge this PR. - DONE
  2. Run rails dqt_to_trs_migration:replace_dqt_trn_requests_table_with_trs_trn_requests rake task. - DONE
  3. PR to add new table to dataform - DONE
  4. Remove dqt_trn_requests table and any use of it. - [AQTS-688] Drop DQTTRNRequest model and table as it is no longer in use #2556
  5. PR to remove dqt_trn_requests to to dataform.

Copy link

github-actions bot commented Dec 6, 2024

@Hassanmir92 Hassanmir92 force-pushed the AQTS-688-dev-code-architecture-update-new-trs-trn-request-table branch from 674dccd to fcfc043 Compare December 10, 2024 11:05
@Hassanmir92 Hassanmir92 force-pushed the AQTS-688-dev-code-architecture-update-new-trs-trn-request-table branch from fcfc043 to 37f0d4d Compare December 10, 2024 11:06
Copy link
Contributor

@syed87 syed87 left a comment

Choose a reason for hiding this comment

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

lgtm

@Hassanmir92 Hassanmir92 merged commit 559d04b into main Dec 10, 2024
14 checks passed
@Hassanmir92 Hassanmir92 deleted the AQTS-688-dev-code-architecture-update-new-trs-trn-request-table branch December 10, 2024 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants