- Issue created by @cosmicdreams
- π©πͺGermany jurgenhaas Gottmadingen
Thanks @cosmicdreams for that proposal, you're absolutely right that using drush is much preferred compared to using the drupal script. And that's true for all environments, not only for ddev.
The reason why we've done it that way back when we introduced the recipes was, that only Drush 13 supports the recipe command and that version wasn't even released back then.
That's different now, but I'm not sure how many installations of Drush 12 are still around in the Drupal 10.3 or later environments.
Maybe we should just extend the documentation and show both options? The composer command stays the same anyway, so it's only about applying the recipe could be done with the script or with Drush. The user can then choose.
The cash refresh, though, sounds a bit odd. If that's really needed, I'm not sure. If anything, that sounds like a ddev issue and I would keep that out of the ECA documentation.
- πΊπΈUnited States cosmicdreams Minneapolis/St. Paul
In my view, the cache clear was needed.
Give it a go, maybe the issue is just a me thing
- πΊπΈUnited States cosmicdreams Minneapolis/St. Paul
In this time when we are in between, with some people using an older version of drush and some using the newest, perhaps we should provide both commands and document what versions they are compatible with.
I can go through the documentation pages and make the changes myself if that helps.
- π©πͺGermany jurgenhaas Gottmadingen
Thanks @cosmicdreams for your suggestions, that all sounds great.
I can go through the documentation pages and make the changes myself if that helps.
Fortunately, we don't have to edit all the samples individually. There is a twig template which is used to create all that automatically. You can open an MR here to make that change once, and then we only have to apply that to the documentation, when it's done. You can find that template at
eca/modules/development/templates/docs/library.md.twig
- Merge request !449Expand documentation per recipe to include guidance for new and old Drush. β (Merged) created by cosmicdreams
- Status changed to Needs review
6 months ago 1:53pm 18 September 2024 - Status changed to Needs work
5 months ago 8:43am 19 September 2024 - π©πͺGermany jurgenhaas Gottmadingen
This looks like a good starting point. I've left a couple of comments, though.
I'm wondering, if the required instructions get so verbose, that we may consider a separate chapter for the instructions and simply link to that from each sample page?
- Status changed to Fixed
3 months ago 12:23pm 8 December 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- π©πͺGermany jurgenhaas Gottmadingen
For some reason, this got closed without actually merging the MR. Must have been a manual error on my side. I've just merged it now and will update the ECA Guide asap.