search

localgovdrupal / localgov_workflows

Default editorial workflow for LocalGov Drupal content.
GNU General Public License v2.0
0 stars 1 forks source link

Errors when installing site from existing configuration #64

Open Polynya opened 11 months ago

Polynya commented 11 months ago

Relates to https://github.com/localgovdrupal/localgov_workflows/issues/63

Polynya commented 10 months ago

Thanks for investigating this. For me, without LocalGov Demo, when I run site install it adds the correct content types to the workflow and scheduled transitions.

If I install LocalGov Demo, export the config and run site install then I first hit this error: Error: Call to a member function setSyncing() on null in Drupal\Core\Config\Entity\ConfigEntityStorage->importDelete()

This is resolved by the patch here: https://www.drupal.org/project/drupal/issues/3051453

It doesn't solve the underlying issue, and only continues to more errors e.g. Unable to determine class for field type '' found in the 'field.field.' configuration

finnlewis commented 9 months ago

Just looking at this at Merge Monday again. @ekes thinks we do need to incude the hook_install, but we also want to be able to install from existing config, and react differently. We'll need to give more time to this.

finnlewis commented 9 months ago

@stephen-cox any chance you can make some time to look at this in more depth?

finnlewis commented 9 months ago

Would hook_modules_installed be of use? https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Extension%21module.api.php/function/hook_modules_installed/9.0.x#:~:text=function%20hook_modules_installed&text=Perform%20necessary%20actions%20after%20modules,the%20module%20actually%20being%20installed.

stephen-cox commented 9 months ago

Happy to take a look sometime, but might be a while before I get to it.

finnlewis commented 8 months ago

Just noting that no one has had time to look at this lately.

Keen to figure out the best way to support installing from config and everything else!