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

chore(EMS-3901): data migration - no pdf iterations #3193

Merged
merged 15 commits into from
Oct 25, 2024

Conversation

ttbarnes
Copy link
Contributor

@ttbarnes ttbarnes commented Oct 16, 2024

Introduction ✏️

This PR introduces a new, small data migration script so that when we go live with main-iterations ("No PDF iterations"), any applications that have a status of "in progress" will have versions and migration flags added, as well as 1x new field requestedCreditLimit in the Policy table.

Resolution ✔️

  • Extract API VERSIONS constants into separate objects, add new VERSION_3 object.
  • Extract the existing data migration executeSqlQuery and connectToDatabase functions into a parent directory so that they can be used by any data migration script.
  • Create version-2-to-version-3 data migration script and README.
  • Update the data-migration NPM command.
  • Add new migratedV2toV3 field.

Miscellaneous ➕

  • Minor documentation and logging improvements.

@ttbarnes ttbarnes added documentation Improvements or additions to documentation tech Technical debt/improvements chore Chore labels Oct 16, 2024
@ttbarnes ttbarnes self-assigned this Oct 16, 2024
@ttbarnes ttbarnes marked this pull request as ready for review October 16, 2024 16:04
Copy link

@ttbarnes ttbarnes merged commit 13d128a into main-iterations Oct 25, 2024
181 of 182 checks passed
@ttbarnes ttbarnes deleted the chore/EMS-3901 branch October 25, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore documentation Improvements or additions to documentation tech Technical debt/improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants