- Issue created by @eojthebrave
- Merge request !62Remove instructions for downloading a module or theme using the Update... β (Merged) created by eojthebrave
- Status changed to Needs review
4 months ago 8:42pm 15 August 2024 - πΊπΈUnited States eojthebrave Minneapolis, MN
Here's my first attempt at updating this. I remove the content about using the Update Manager UI. Made the use of Composer more prominent. And opted to move instructions about how to start using Composer with a project that was originally created from the .tar.gz files to the page about dealing with manual installations and off the Composer instructions page so that it's closer to where someone would actually use it. And, will make it easier for us to potentially just delete that content all together in the future.
- Status changed to Needs work
4 months ago 9:22pm 15 August 2024 - πΊπΈUnited States Amber Himes Matz Portland, OR USA
I agree that the manual downloading instructions should be removed in favor of using Composer.
I reviewed the MR and caught some typos and commas. After these changes are made, I think this is good to go.
- Status changed to Needs review
4 months ago 9:42pm 15 August 2024 - Status changed to RTBC
4 months ago 10:09pm 15 August 2024 - π³πΏNew Zealand quietone
Thanks for working on this!
Adding related core issue.
- Status changed to Needs work
4 months ago 4:35pm 19 August 2024 - πΊπΈUnited States eojthebrave Minneapolis, MN
Another option would be to remove the content about using .tar.gz files completely. And focus on Composer only. Which seems like it might be a good idea given the issue linked by @quietone. π± [meta] Deprecate tarballs, because they are incompatible with Composer-managed dependencies, Automatic Updates, Project Browser, and release managers' health Active . Which is recommending that for core at least the option to install Drupal via .tar.gz be removed.
And while the option to use .tar.gz files for contrib might still exist it is not the recommended way so we shouldn't be promoting it.
I'm going to work on an update to this MR that removes the manual install instructions completely. Anyone have objections?
- π©π°Denmark ressa Copenhagen
I agree @eojthebrave, it makes sense to remove content about using .tar.gz files completely and focus on Composer only, since the option to install tarballs was removed in Drupal 10.4 and Drupal 11 via π Remove adding an extension via a URL Fixed .
- Status changed to Needs review
4 months ago 4:54pm 20 August 2024 - πΊπΈUnited States eojthebrave Minneapolis, MN
Okay I've made significant updates to the MR and it's ready for another review. This update:
- Updates the pages about applying module and theme updates to remove the use of the Update Manager UI and only recommend using Composer. The Update Manager UI isn't really compatible with Composer, and long term we should stop recommending it and wait for Automatic Updates. This also required updating some screenshots.
- Remove the content about installing core from a .tar.gz file completely.
- Update the content about manually installing a module or theme that was previously about using a .tar.gz file to be about installing custom modules or themes that are no available via Composer.
- Status changed to Needs work
4 months ago 7:06pm 20 August 2024 - πΊπΈUnited States Amber Himes Matz Portland, OR USA
Thanks for the update, @eojthebrave! I've reviewed it again and just have some copy edit suggestions.
Also, I just used the GitLab WebIDE to search for links to the deleted file, install-manual, but I'm not sure I trust that 100%. So just wanted to make sure that you searched and removed links to the file you deleted.
- Status changed to RTBC
4 months ago 7:51pm 20 August 2024 - πΊπΈUnited States eojthebrave Minneapolis, MN
Thanks for the review and copy edits.
Also, I just used the GitLab WebIDE to search for links to the deleted file ...
Yes, I did a search through all the English language source files and the string `install-manual` is no longer present in any of them.
-
eojthebrave β
committed bd18ed5f on 11.x
Issue #3468377 by eojthebrave, amber himes matz: Update (or remove)...
-
eojthebrave β
committed bd18ed5f on 11.x
- Status changed to Fixed
4 months ago 8:24pm 20 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.