Skip to content

Replace term "storage apps" with "projects" - #45612

Merged
maureensturgeon merged 2 commits into
stagingfrom
maureen/LP-2272-storage-apps-cleanup-2
Apr 5, 2022
Merged

Replace term "storage apps" with "projects"#45612
maureensturgeon merged 2 commits into
stagingfrom
maureen/LP-2272-storage-apps-cleanup-2

Conversation

@maureensturgeon

@maureensturgeon maureensturgeon commented Apr 1, 2022

Copy link
Copy Markdown
Contributor

The table pegasus.storage_apps has been migrated to dashboard.projects (#45480), however there are still many references to the term "storage apps".

In this PR

  • an alias is created for the active record models ChannelTokens and Backpacks to rename storage_app_id to project_id
  • the class StorageApps has been renamed to Projects and all references have been updated

Links

Testing story

Tested some project scenarios locally

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 changed the base branch from staging to maureen/LP-2272-storage-apps-cleanup April 1, 2022 18:27
Base automatically changed from maureen/LP-2272-storage-apps-cleanup to staging April 1, 2022 20:53
@maureensturgeon maureensturgeon changed the title storage_apps clean up Replace term "storage apps" with "projects" Apr 1, 2022
@maureensturgeon
maureensturgeon marked this pull request as ready for review April 1, 2022 22:37
@maureensturgeon
maureensturgeon requested a review from a team April 1, 2022 22:37
@maureensturgeon
maureensturgeon merged commit 5ce06db into staging Apr 5, 2022
@maureensturgeon
maureensturgeon deleted the maureen/LP-2272-storage-apps-cleanup-2 branch April 5, 2022 00:07
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