Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Broken Links #2246

Closed
wants to merge 1 commit into from
Closed

Conversation

justindhillon
Copy link

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 ⭐

Copy link

netlify bot commented Feb 18, 2024

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 2d9b512
🔍 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
Copy link
Collaborator

plowsof commented Feb 25, 2024

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 7ec2e0c

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
Copy link
Collaborator

plowsof commented Jul 14, 2024

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)

@plowsof plowsof closed this Jul 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants