- Issue created by @quietone
- 🇳🇿New Zealand quietone
The documentation URL that were 404 should be fixed, either by a new redirect or this MR. As tests shouldn't rely on d.o, changing those does not matter.
However, I didn't do a great deal of research into the following two. However, these are tests so, if someone thinks they need to change I would prefer that in a separate issue. I think it better to fix, at least, the documentation link in this issue.
- Status changed to Needs review
11 months ago 3:25am 22 January 2024 - Status changed to Needs work
10 months ago 5:45am 24 January 2024 I verified the URLs given under 'Proposed resolution' for 404 errors. But, can still see the error is present on those URLs.
https://www.drupal.org/contribute/core-maintainers →
https://www.drupal.org/foo/bar?foo=bar&bar=baz&baz#foo →
https://www.drupal.org/sample →
https://www.drupal.org/node/123 →
https://www.drupal.org/node/3395641 →
https://www.drupal.org/node/3301971 →URLs redirection is now working fine for below URLs which are not throwing an 404 errors
https://www.drupal.org/docs/upgrading-drupal/drupal-8-and-higher →
https://www.drupal.org/docs/upgrading-drupal/upgrading-from-drupal-8-or-... →- Status changed to Needs review
10 months ago 6:44am 24 January 2024 - 🇳🇿New Zealand quietone
@asawari, thanks for testing. I think there is a bit of confusion here. Most of the links fixed are fixed in the code base only. You need to check out the MR and test the links that are changed in code. For example, https://www.drupal.org/contribute/core-maintainers → is changed in the MR.
Also, there are a few that are being fixed in other issues. Those other issues are identified in the Issue Summary. The 'Done' in the Issue Summary means that the fix is either in the MR for this issue, a re-direct or another issue.
Cheers,
- First commit to issue fork.
- Status changed to Needs work
10 months ago 1:12pm 24 January 2024 - 🇮🇳India Meeni_Dhobale
Tried to fix
wget https://www.drupal.org/files/projects/drupal-x.y.z.tar.gz
URL to download Drupal. - First commit to issue fork.
- 🇮🇳India arunkumark Coimbatore
The wget install core path is invalid. The valid path is
https://www.drupal.org/files/projects/drupal-x.y.z.tar.gz
. Update the MR to fix the issue. - Status changed to Needs review
10 months ago 2:36pm 24 January 2024 - Status changed to Needs work
10 months ago 3:20pm 25 January 2024 - 🇺🇸United States smustgrave
From what I can tell not all the links have been addressed.
Just picked 1 link https://www.drupal.org/SA-CONTRIB-2019-02-02 → and that's still in the repo
- Status changed to Needs review
10 months ago 8:08am 26 January 2024 - 🇳🇿New Zealand quietone
Please read the Issue Summary, it includes the resolution for that URL.
- 🇺🇸United States smustgrave
Read the issue summary and it's not clear what is being addressed here.
- 🇳🇿New Zealand quietone
The proposed resolution has two sections. One for the links that are fixed as a result of this issue and a 'no change' section to indicate the URLs that are not to be changed. I have added more text to the issue summary.
- Status changed to RTBC
10 months ago 6:50pm 28 January 2024 - 🇺🇸United States smustgrave
Thank you @quietone for clarify what links were fixed here. Searched the repo for those and confirmed the links mentioned have been addressed
- Status changed to Needs review
10 months ago 11:53pm 2 February 2024 - 🇸🇰Slovakia poker10
This URL https://www.drupal.org/contribute/core-maintainers → is also in D7 maintainers.txt file. I think we should set up/restore that redirect, instead of fixing that in core repo. Because evidently the link (or at least a redirect) was working some time ago (https://web.archive.org/web/20230801000000*/https://www.drupal.org/contr...). Moving back to NR to consider this, thanks!
- 🇳🇿New Zealand quietone
@poker10, thanks! I have removed that changed and made the redirect.
- Status changed to RTBC
10 months ago 9:56am 7 February 2024 - Status changed to Fixed
9 months ago 2:51pm 29 February 2024 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.