- First commit to issue fork.
- Assigned to kunal_sahu
- Status changed to Needs work
over 1 year ago 12:27pm 23 February 2023 - 🇮🇳India kunal_sahu Karnataka
Hi , Your patch seems to fail for me.
Moving it to Needs work.
PS C:\xampp\htdocs\drupal-9.5\web\modules\contrib\mailsystem-2054871> git apply .\broken-link-2054871-2.patch error: patch failed: README.html:1 error: README.html: patch does not apply error: patch failed: README.markdown:7 error: README.markdown: patch does not apply error: patch failed: README.txt:5 error: README.txt: patch does not applywill provide a patch for the same. Thanks
- @kunal_sahu opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 12:38pm 23 February 2023 - First commit to issue fork.
- 🇬🇧United Kingdom schillerm
Hi, reviewed the MR and noticed that the numbering was wrong in README.txt. So I fixed that and pushed the change up .
- Status changed to RTBC
over 1 year ago 7:30pm 8 March 2023 - 🇮🇳India saket-001
MR 6 is working fine with correct numbering i moving status to RTBC.
- Status changed to Needs work
over 1 year ago 11:49pm 8 March 2023 - 🇺🇸United States tr Cascadia
No, a bunch of links are still broken. Note mail.inc no longer exists, so all those links to the mail.inc api documentation don't work. Likewise, the links to the api pages should not end in /8 as that specifies an exact core version. If you leave that off it will redirect to the page for the current core version. And why delete the link to the screenshot? That was and still is useful documentation. Correct the link instead of deleting it! If the image is no longer on drupal.org, then take a screenshot, upload it, and link to that new screenshot. Don't just throw away valuable information ...