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

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

Merged
merged 2 commits into from
Apr 5, 2022

Conversation

maureensturgeon
Copy link
Contributor

@maureensturgeon maureensturgeon commented Apr 1, 2022

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
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.

None yet

2 participants