Problem/Motivation

On /en/about-umami, I noticed a link to https://www.drupal.org/docs/8/umami-drupal-8-demonstration-installation-.... When I follow the link, it redirects to https://www.drupal.org/docs/umami-drupal-demonstration-installation-profile.

We should update that link, and others, in order to avoid the confusing redirect.

Proposed resolution

Search for drupal.org/docs in the Umami profile:

grep -ri drupal.org/docs core/profiles/demo_umami

Currently, I find it in two CSV files (for the English and Spanish "About Umami" pages) and in README.txt.

Load each link in a browser. If it redirects you to a different URL, then update the code.

Make sure to update both the link text and the href attribute.

Remaining tasks

User interface changes

This issue will update the links on the English and Spanish "About Umami" pages.

API changes

None

Data model changes

None

Release notes snippet

N/A

Issue fork drupal-3277309

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:

Comments

benjifisher created an issue. See original summary.

phjou’s picture

I start working on this during DrupalCon Portland.

phjou’s picture

Issue tags: +GiftofOpenSource

phjou’s picture

Status: Active » Needs review
benjifisher’s picture

@phjou, thanks for fixing this issue!

I reviewed the changes, and they look right.

I checked the link in the README by looking at the issue fork: https://git.drupalcode.org/issue/drupal-3277309/-/tree/3277309-update-li.... The link there works.

I installed the demo_umami profile locally and visited the English and Spanish versions of the "About Umami" page. The links worked.

The first test failed with "Fetch Error". I think that is an infrastructure problem, not a problem with the MR. I queued another test.

Unless the testbot disagrees, this issue is fully RTBC.

benjifisher’s picture

Status: Needs review » Reviewed & tested by the community

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

markie’s picture

assisted with this issue yesterday during the mentored contribution. Thanks for the quick review.

  • lauriii committed 04c1b7f on 10.0.x
    Issue #3277309 by phjou, mradcliffe, benjifisher, markie: Update links...

  • lauriii committed d781860 on 9.4.x
    Issue #3277309 by phjou, mradcliffe, benjifisher, markie: Update links...

  • lauriii committed ffc6e14 on 9.3.x
    Issue #3277309 by phjou, mradcliffe, benjifisher, markie: Update links...

  • lauriii committed 5ffd6ae on 9.5.x
    Issue #3277309 by phjou, mradcliffe, benjifisher, markie: Update links...
lauriii’s picture

Version: 9.4.x-dev » 9.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 04c1b7f and pushed to 10.0.x. Cherry-picked to 9.5.x, 9.4.x and 9.3.x because Umami is internal. Thanks!

Status: Fixed » Closed (fixed)

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