The 'About Umami' page in Umami has a link that goes to the initiative issue page rather than the documentation page.

Remove the following link: https://www.drupal.org/project/ideas/issues/2847582

And replace it with: https://www.drupal.org/docs/8/umami-drupal-8-demonstration-installation-...

CommentFileSizeAuthor
#5 replace-the-link-in-the-body-field-on-umami's-about-page-2999014-5.patch1.21 KBrajeevgole
#3 replace-the-link-in-the-body-field-on-umami's-about-page-2999014-12770369.patch1.21 KBrajeevgole

Comments

kjay created an issue. See original summary.

rajeevgole’s picture

rajeevgole
He/Him
Primary language English
commented
Assigned: Unassigned » rajeevgole

I am working on it.

rajeevgole’s picture

rajeevgole
He/Him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new replace-the-link-in-the-body-field-on-umami's-about-page-2999014-12770369.patch1.21 KB
rajeevgole’s picture

rajeevgole
He/Him
Primary language English
commented
Assigned: rajeevgole » Unassigned
rajeevgole’s picture

rajeevgole
He/Him
Primary language English
commented
StatusFileSize
new replace-the-link-in-the-body-field-on-umami's-about-page-2999014-5.patch1.21 KB
eli-t’s picture

eli-t
he/him
Primary language English
Location Manchester
commented

This would be a good issue for novice review at the code sprint. Suggest manually reviewing the patch, and installing on SimplyTest.me with the patch applied to make sure the about page link is updated and points to the documentation page required.

casaran’s picture

casaran commented
Status: Needs review » Reviewed & tested by the community

Tested on SimplyTest.me. Seems to work.

dimitri1972’s picture

dimitri1972 commented

Tested on ddev and worked for me. Good job !

  • lauriii committed 8400c7a on 8.7.x 
    Issue #2999014 by rajeevgole, kjay, Eli-T, casaran, dimitri1972: Replace...

  • lauriii committed a5b6e87 on 8.6.x 
    Issue #2999014 by rajeevgole, kjay, Eli-T, casaran, dimitri1972: Replace...
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Status: Reviewed & tested by the community » Fixed

Committed 8400c7a and pushed to 8.7.x. Also cherry-picked to 8.6.x. Thanks!

Status: Fixed » Closed (fixed)

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