Problem/Motivation

#3232568: Switch Varbase Entity Definition Update Manager class to extend the vardot/entity-definition-update-manager class library

Steps to reproduce

Given that on the install of Varbase API module
When it needs to update entity definitions on install or updates
Then the old namespace was not right and the arguments had changed for Drupal 9
And the Support for automatic entity updates has been removed
https://www.drupal.org/node/3034742

Proposed resolution

Switch to EntityDefinitionUpdateManager class from Vardot/entity-definition-update-manager

Remaining tasks

  • ✅ File an issue about this project
  • ✅ Switch to EntityDefinitionUpdateManager class
  • ✅ Testing to ensure no regression
  • ✅ Automated unit/functional testing coverage
  • ✅ Developer Documentation support on feature change/addition
  • ➖ User Guide Documentation support on feature change/addition
  • ✅ Code review from 1 Varbase core team member
  • ✅ Full testing and approval
  • ✅ Credit contributors
  • ✅ Review with the product owner
  • ✅ Release varbase-9.0.2, varbase_api 9.0.2

User interface changes

  • None

API changes

  • None

Data model changes

  • None

Comments

RajabNatshah created an issue. See original summary.

Rajab Natshah’s picture

Rajab Natshah’s picture

Issue summary: View changes

  • RajabNatshah committed b5abf36 on 9.0.x
    Issue #3232612: Switch Varbase API  to use Entity Definition Update...
Rajab Natshah’s picture

Assigned: Rajab Natshah » Mohammed J. Razem
Issue summary: View changes
Status: Active » Needs review
Issue tags: +varbae-9.0.2, +varbase_api-9.0.2
Rajab Natshah’s picture

Assigned: Mohammed J. Razem » Unassigned
Rajab Natshah’s picture

Issue summary: View changes
Rajab Natshah’s picture

Status: Needs review » Fixed
Rajab Natshah’s picture

Issue summary: View changes
Rajab Natshah’s picture

Issue summary: View changes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Rajab Natshah’s picture

Issue summary: View changes
Rajab Natshah’s picture

Issue summary: View changes