Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3587769

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

sivaji_ganesh_jojodae created an issue. See original summary.

sivaji_ganesh_jojodae opened merge request !15624

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Active » Needs work

Left some comments on the MR - nice one

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented

Thanks for the feedback. I’ve refactored the Hook classes by splitting them in a manner similar to the Update and System modules. However, I’m not entirely confident that all methods have been placed in the most appropriate files. There are still a few methods where the correct separation is unclear. Additionally, I’ve noticed that the Workspaces hook classes follow a different naming convention.