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

3.x Rename the entity base table #43

Open
wants to merge 8 commits into
base: 3.0.x
Choose a base branch
from

Conversation

WengerK
Copy link
Member

@WengerK WengerK commented Jul 5, 2024

💬 Describe the pull request

Rename base table from backerymails_sended_mail to backerymails_sent_mails

  • Update existing views
  • Update the entity
  • Migrate data from old to new table
  • Drop table when everything has been migated
  • Upgrade path coverage

🗃️ Issues

This pull request is related to :

🔢 To Review

Steps to review the PR:

  1. drush updatedb -y

📸 Pay attention to

  • All existing data in the previous table should be migrated
  • People who install the module for the 1st time should not need the updatedb, therefor nothing should break when the previous table does not exists
  • The Backerymail views should be updated with the new base-table name

@WengerK WengerK force-pushed the 30x/16-refactoring-entity-base-table- branch from d0abd71 to 78e45dd Compare July 5, 2024 15:06
@WengerK WengerK changed the title 2.x Rename the entity base table 3.x Rename the entity base table Jul 5, 2024
@WengerK WengerK force-pushed the 30x/16-refactoring-entity-base-table- branch from 78e45dd to 95ed1ad Compare July 5, 2024 15:20
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.

[Drupal - Backerymails] - Refactoring & Migrations Table name incorrect
1 participant