-
Notifications
You must be signed in to change notification settings - Fork 46
Define install tasks for choosing the site template and add-ons #151
Define install tasks for choosing the site template and add-ons #151
Conversation
Oooh, I think I see the problem you mention, @narendra-drupal. It's not in scope here, but the default content in Starshot is in the English language. The thing we probably need to do is move the demo content to its own recipe, and have it explicitly install in English only. |
I tested this at the command line and one problem we have is that, if you run
It should be |
The closer I look at this, the more troublesome it becomes... I think what we might need to do here is actually not use hook_install_tasks_alter(), but rather, create a new install task as part of the installer that allows you to choose a recipe, and then ensure that recipe is applied by |
After discussion with @timplunkett, I think we should change the approach here. We should not use
To put it another way: you cannot skip the installer, and you cannot install the site from a recipe instead of using the |
Done |
3d75726
to
83734b2
Compare
While working on this I found that if we try to install starshot in other language it gives error
Drupal\Core\DefaultContent\InvalidEntityException: recipes/starshot/content/user/c6fc2154-25c0-4b9d-ad4d-8d04a6a52f05.yml: preferred_langcode.0.value=The value you selected is not a valid choice.||preferred_langcode.0=The value you selected is not a valid choice.||preferred_admin_langcode.0.value=The value you selected is not a valid choice.||preferred_admin_langcode.0=The value you selected is not a valid choice. in Drupal\Core\DefaultContent\Importer->importContent() (line 113 of Sites/starshot/web/core/lib/Drupal/Core/DefaultContent/Importer.php).