Problem/Motivation

   * @see https://www.drupal.org/project/automatics_updates/issues/3312619#comment-14801308

is a 404.

It should have been fixed in #3553528: Fix documenation URLs that are 404, part 1 but that made a typo in the project name.

Steps to reproduce

Proposed resolution

Change the URL to the canonical node path.

Remaining tasks

MR
Review
Commit

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3569778

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

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue tags: +11.4.0 release priority

Adding priority because this should be the last child issue. And once committed all the URLs in docs should, at least, not be a 404.

quietone’s picture

quietone commented
Component: package_manager.module » documentation
garvitasakhrani’s picture

garvitasakhrani commented
Assigned: Unassigned » garvitasakhrani

I am working on it.

garvitasakhrani opened merge request !14510

garvitasakhrani’s picture

garvitasakhrani commented
Assigned: garvitasakhrani » Unassigned
Status: Active » Needs review

Fixed the broken documentation link in TestSandboxManager::__sleep by correcting the project name in the URL.
Changes are present in the issue fork branch 3569778-fix-link-in.
Please review. Thanks!

dcam’s picture

dcam commented

Is there any reason why the URL shouldn't be the canonical https://www.drupal.org/node/3312619#comment-14801308 or the even shorter https://drupal.org/i/3312619#comment-14801308?

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

angel_devoeted’s picture

angel_devoeted commented

Updated the MR to use the canonical node URL (/node/3312619#comment-14801308) as suggested by @dcam.

dcam’s picture

dcam commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

This is a one-line documentation change to fix a broken link. I verified that it goes to the correct place now. There isn't much else to say about it. It looks good to me.

larowlan closed merge request !14510

  • larowlan committed b6c5ebee on 11.3.x 
    fix: #3569778 Fix link in \Drupal\Tests\package_manager\Kernel\...

  • larowlan committed 9c74d384 on 11.x 
    fix: #3569778 Fix link in \Drupal\Tests\package_manager\Kernel\...

  • larowlan committed 79bb0dc6 on main 
    fix: #3569778 Fix link in \Drupal\Tests\package_manager\Kernel\...
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Version: main » 11.3.x-dev
Status: Reviewed & tested by the community » Fixed
Issue tags: -11.4.0 release priority

Committed to main and backported to 11.x. As this is a bugfix with zero disruption, also backported to 11.3.x.

Thanks folks here's a puppy 🐕️

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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