Problem/Motivation

Right now, we show the instructions for how to install Drush, even if you already have Drush. That's kind of goofy. Now that instructions are generated on the backend, we can detect if Drush is installed and only show the instructions if necessary.

CommentFileSizeAuthor
#3 drush-after.png49.83 KBrajdip_755
#3 drush-before.png99.67 KBrajdip_755

Issue fork project_browser-3458287

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

phenaproxima created an issue. See original summary.

phenaproxima opened merge request !535

rajdip_755’s picture

rajdip_755
Primary language Bengali
Location kolkata
commented
StatusFileSize
new drush-before.png99.67 KB
new drush-after.png49.83 KB

Hi @phenaproxima, I have applied your patch and it's applied successfully with the following setup.

project_browser version: 2.0.0-alpha2
drupal version: 11.0.1

And after applying the patch the command for installing drush is not showing anymore. I'm attaching here both the screenshots of before and after applying the patch respectively.

I'm moving it to RTBC.

Thanks !

rajdip_755’s picture

rajdip_755
Primary language Bengali
Location kolkata
commented
Status: Active » Reviewed & tested by the community

chrisfromredfin made their first commit to this issue’s fork.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Reviewed & tested by the community » Fixed

Confirming that the test failures were flaky/intermittent, so merging.

Status: Fixed » Closed (fixed)

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