Thank you for all of the work on the user guide.

I discovered an issues with https://www.drupal.org/docs/user_guide/en/extend-theme-install.html

Using drush on a new drupal 8.6.10 install I get the following error when using the "drush dl" to get a new theme:
web$ drush dl bootstrap
In Application.php line 267:
dl has been deprecated. Please build your site using Composer. Add new projects with composer require drupal/[project-name]. Use https://www.drupal.org/project/composer_generate to build a composer.json which represents the enabled modules on your site.

CommentFileSizeAuthor
#7 3035394-7-update-video.patch583 bytesAmber Himes Matz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

CProfessionals created an issue. See original summary.

jhodgdon’s picture

Status: Active » Postponed (maintainer needs more info)

I don't see where on that page it says to use drush dl? I think we removed all references to drush dl in an earlier issue, for exactly the reason you are referencing. ??? Maybe you were looking at an old version of the User Guide?

eojthebrave’s picture

Title: Installing theme » [video] Installing theme
Assigned: Unassigned » eojthebrave
Status: Postponed (maintainer needs more info) » Active

I'm assuming this refers to the video content, like with the other issue that @CProfessionals posted here #3035395: [video] Manual Theme install as third option, in fact these two issues seem like they're kind of intertwined. For example, that one has a comment https://www.drupal.org/project/user_guide/issues/3035395#comment-12985436 which appears to answer the question above regarding reference to `drush dl` in the content.

jhodgdon’s picture

Component: User Guide content » Videos
Amber Himes Matz’s picture

The updated video URL is https://www.youtube.com/watch?v=UFgddj0F_bU

I can do a patch shortly or defer to @eojthebrave who is currently assigned the task.

Amber Himes Matz’s picture

I'll work on a patch this morning.

Amber Himes Matz’s picture

Assigned: Amber Himes Matz » Unassigned
Status: Active » Needs review
FileSize
583 bytes

Here is a patch that updates the embedded video to the new updated one.

eojthebrave’s picture

Status: Needs review » Fixed

👍 Thanks @Amber Himes Matz

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.