Skip to content

Migration to drop project_versions temp view - #47208

Merged
maureensturgeon merged 1 commit into
stagingfrom
maureen/LP-2415-remove-project-versions-temp-view
Jul 18, 2022
Merged

Migration to drop project_versions temp view#47208
maureensturgeon merged 1 commit into
stagingfrom
maureen/LP-2415-remove-project-versions-temp-view

Conversation

@maureensturgeon

@maureensturgeon maureensturgeon commented Jul 13, 2022

Copy link
Copy Markdown
Contributor

This is the clean-up from the project_versions -> project_commits DB table rename. This is the migration to drop the temporary view.

Older PRs for context:

Links

Testing story

Checked locally that the temporary view is dropped. Searched through the codebase to make sure that no references to project_versions table were left.

Also left commits on CSA level after dropping the view to ensure creating and fetching commits still works as expected.

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@maureensturgeon
maureensturgeon marked this pull request as ready for review July 13, 2022 17:44
@maureensturgeon
maureensturgeon requested a review from a team as a code owner July 13, 2022 17:44
@maureensturgeon
maureensturgeon requested review from a team July 13, 2022 17:44
@maureensturgeon
maureensturgeon force-pushed the maureen/LP-2415-remove-project-versions-temp-view branch from 664b8a0 to 695f0e4 Compare July 14, 2022 00:26
@maureensturgeon
maureensturgeon merged commit 490ea91 into staging Jul 18, 2022
@maureensturgeon
maureensturgeon deleted the maureen/LP-2415-remove-project-versions-temp-view branch July 18, 2022 16:21
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