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

Launchpad: fix broken links #78

Open
jugmac00 opened this issue Jun 6, 2024 · 7 comments
Open

Launchpad: fix broken links #78

jugmac00 opened this issue Jun 6, 2024 · 7 comments
Assignees
Labels
good first issue An ideal task to start with help wanted We welcome community help with this issue review Review pre-existing documentation for quality and consistency size 2 Fix a known documentation issue (size 2/8) update Update potentially outdated instructions, commands, or version numbers

Comments

@jugmac00
Copy link
Member

jugmac00 commented Jun 6, 2024

Documentation: https://documentation.ubuntu.com/launchpad/en/latest/how-to/
GitHub repo: https://github.com/canonical/launchpad-manual
Contribution guide: https://documentation.ubuntu.com/launchpad/en/latest/contribute-to-our-docs/

For the migration of the documentation, a lot of existing documentation was transferred from other platforms, where no checks were performed before. Some of the added pages were excluded from the new link checks.

Task: remove the exceptions one by one from https://github.com/canonical/launchpad-manual/blob/f410f03e21bb70720c2763225b4313744a09cd4b/custom_conf.py#L112 (e.g.'Trunk/Glue', # needs update), run the link checks (make linkcheck), fix the broken links, and create a pull request, preferably one for each page to make reviews easier. Only the links marked with needs update need to be considered.

@degville degville added good first issue An ideal task to start with help wanted We welcome community help with this issue review Review pre-existing documentation for quality and consistency size 2 Fix a known documentation issue (size 2/8) update Update potentially outdated instructions, commands, or version numbers labels Jun 6, 2024
@nielsenjared
Copy link

I'm interested in this issue!

@jugmac00
Copy link
Member Author

@nielsenjared Sounds great! Please let us know if you need any support.

@nielsenjared
Copy link

Hey @jugmac00! Where do I find the correct links?

Here's some context:

  • I removed 'Trunk/Glue', ran linkcheck and received an error as outlined above and expected.
  • I found reference to 'Trunk/Glue' in ./explanation/branches.rst (AKA "Where's trunk?")
  • I can't find the 'Trunk/Glue' page in the repository.
  • I can't find the 'Trunk/Glue' page on the new docs https://documentation.ubuntu.com/launchpad/en/latest/
  • I did find the 'Trunk/Glue' page via Google (as well as a a lot of auto repair tutorials) on what I assume is the old doc site https://dev.launchpad.net/Trunk/Glue

Is this the link I need to use? Am I missing something in the repo or the new site?

@nielsenjared
Copy link

Hey @jugmac00!
I think I addressed what I can here. I'm unable to locate the files that correspond to the links below. I tracked down their legacy versions and used that content to cross-reference the new repo. In some instances I was able to find updated/renamed versions of the pages, but not for the following:

Let me know if I missed something or if I can help with anything else.

Cheers!

@jugmac00
Copy link
Member Author

jugmac00 commented Aug 16, 2024

@nielsenjared Thanks so much for your contributed link fixes, and especially thanks for your investigation about the remaining items.

I went through the list and added some comments what I think needs to be done.

https://dev.launchpad.net/Code/Concepts
https://pastebin.ubuntu.com/p/StbrX2vn2v/

  • needs to be imported

https://dev.launchpad.net/Code/HowToUseCodehostingLocally
https://pastebin.ubuntu.com/p/8QkDgWKS4P/

  • needs to be imported

https://dev.launchpad.net/Database/TableRenamePatch
https://pastebin.ubuntu.com/p/svwKxcKdVY/

  • needs to be imported

https://dev.launchpad.net/Debugging
https://pastebin.ubuntu.com/p/NpJwWpwHHH/

  • needs to be imported

https://dev.launchpad.net/JavascriptUnitTesting/MockIo
https://pastebin.ubuntu.com/p/Nqc6S55WXw/

  • needs to be imported

https://dev.launchpad.net/PolicyAndProcess/Accessibility

  • can be deleted, as a revamp of the UI is planned

https://dev.launchpad.net/Soyuz/HowToUseSoyuzLocally

  • can be found under /how-to/use-soyuz-locally/

https://dev.launchpad.net/Translations/Specs/UpstreamImportIntoUbuntu/FixingIsImported/setCurrentTranslation

  • can be removed

https://dev.launchpad.net/Code?action=AttachFile&do=view&target=codehosting.png

  • copy / paste the image over from the old host

‘attachment:TranslationsSchema.dia’; referenced in explanation/engineering-overview-translations

  • seems to be broken already on the old host; please remove

For the items marked with "needs to be imported", I will provide a pastebin link each with the raw rst (from the auto-migration script). These pages need to be integrated into our documentation, and for the time being a warning header should be set which indicates that these pages need to be manually revised, which can be done via...

.. include:: ../includes/important_not_revised.rst

I will provide the pastebin links with the raw migration content as soon as possible.

@nielsenjared
Copy link

Hey @jugmac00! I addressed everything but the files that remain to be imported. Let me know when they're available.

@jugmac00
Copy link
Member Author

jugmac00 commented Aug 19, 2024

@nielsenjared I added the raw rst which was generated by the migration tool. Thanks a lot for all your support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue An ideal task to start with help wanted We welcome community help with this issue review Review pre-existing documentation for quality and consistency size 2 Fix a known documentation issue (size 2/8) update Update potentially outdated instructions, commands, or version numbers
Projects
None yet
Development

No branches or pull requests

3 participants