search

monero-project / monero-site

https://getmonero.org
BSD 3-Clause "New" or "Revised" License
275 stars 384 forks source link

Fix Broken Links #2246

Closed justindhillon closed 2 months ago

justindhillon commented 7 months ago

Summery

In tor_wallet.md, there is a link to https://github.com/monero-project/monero/blob/master/ANONYMITY_NETWORKS.md. But the file has been moved from the root directory to the doc directory. I have fixed the link in every language.

https://github.com/monero-project/monero/blob/master/ANONYMITY_NETWORKS.md --> https://github.com/monero-project/monero/blob/master/docs/ANONYMITY_NETWORKS.md

Support my work

These links were found with link-inspector. If you find this PR useful, give the repo a ⭐

netlify[bot] commented 7 months ago

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
Latest commit 2d9b51248375c1839244270a7d1ab6dabacd97e7
Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/65d25a7ae20ade000876bd1c
Deploy Preview https://deploy-preview-2246--barolo-time-757cf9.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

plowsof commented 7 months ago

the link is correct / works here: https://www.getmonero.org/resources/user-guides/tor_wallet.html . this is possibly an issue with translations not being updated and needs further investigation.

2 years ago the link was fixed with a direct markdown file edit https://github.com/monero-project/monero-site/commit/7ec2e0c9b175f1d7292a4c4f0a298468d591a6bf

this was bad, as only the pot file should have been changed. so weblate will attempt to revert these changes next time it is ran on the pot file hm

plowsof commented 2 months ago

thank you for raising this issue, but i will close this as po4a will copy the changes over to the none en files for us which is easier to review. (todo)