docs: Change edge version links to latest + fix links intended as relative not absolute#3190
docs: Change edge version links to latest + fix links intended as relative not absolute#3190polarathene merged 5 commits intodocker-mailserver:masterfrom jrpear:master
edge version links to latest + fix links intended as relative not absolute#3190Conversation
|
Unrelated comment: I've been going through lots of the code, I appreciate how well commented it is and all of the links to relevant issues, I've come across a lot from @polarathene. Makes me want to work with this project more! |
There was a problem hiding this comment.
Thanks for tackling contributing this update too! ❤️
Original feedback (resolved)
Looks great 😎 I've been a bit laxed on doc reviews these days and not enforced conventions introduced for consistently managing links.
The preference was to always have doc links use references with relative paths and prefix those references with docs-, usually bundling all those at the bottom of each docs page, but over time some have been co-located closer to the content which is ok too.
I've changed your latest-docs to docs-latest to stick with that convention.
You just need to update the workflow file so we also set the canonical link to latest instead of edge 👍
Whoops, just realized I made that a little bit annoying for you by committing the suggestion myself, sorry about that 😬 Remember to pull this branch first before committing a new change!
Fantastic, thanks for catching those! 😀
I don't have rights to do either of those AFAIK. @wernerfred might? Otherwise you'll need to wait on @georglauterbach I think, who is presently on vacation (and will hopefully make a best effort to remember they're on vacation, and wait until they've returned as there is no rush 😛 ).
We certainly try! 💪 It's something I tend to embrace a bit too excessively at times for the benefit of future maintainers (and sometimes future me who forgets 😂 ) as there is quite a bit of niche knowledge and decisions in the project. I have gotten a bit more lazy with formatting merge commit messages on PRs, so
We would absolutely love any help you can spare time for! ❤️ Contributing docs and assisting in issues as you've been doing is brilliant :) Something maintainers often lack time for is improving the docs and UX (regarding what it's like for new users to get started). There's the shell scripts that glues most of the project together, but I can't say that's particularly enjoyable 😅 or researching parts of the project to troubleshoot / improve something (scripts aside). I know I've sunk a tonne of time going down rabbit holes on niche topics just to fix some config settings 😛 |
|
Cheers, I'll wait for another approval for this change before merging. |
|
Yeah, I believe you're right about the purpose of canonical links. Also the old versions of the docs will still have |
I have a strange fondness for bash haha. Before I found this project, I spent probably 50 hours trying to create something which was nearly identical to dms with bash, systemd, and systemd-machined. |
That's a pretty good point 😅 We could do that in a separate PR I think (keep that noise separate from main focus of this PR). (EDIT: As noted in 2nd section, this is probably not a PR we could review properly 😞 ) That reminds me of this issue with I was working on enabling manual trigger in the CI workflow but didn't get around to finishing it. At the same time, I'm kind of tempted to archive our older versions of docs at some point as we can't realistically support them the older they get. Also I don't think the preview deployments are ever routinely removed either and I probably should look into sorting that out 😬
I'm a bit concerned about the diff size. It's probably not something I can review properly, and we can only trust ignoring large diffs from maintainers 😅 I could perhaps instead run a script to apply the change myself. (EDIT: This is probably the best approach) Alternatively if we improve the workflow to support manual triggering of builds, it could be done that way too (probably not a good idea as running a newer version of the docs generator will probably break old docs with deprecated syntax / features, I know that happened at some point).
Oh wow! You'd be very welcome to help with the scripts if you're comfortable 😁 The other two active maintainers are very good with bash, and we can all advise on the project if you have any questions. |
Nope sorry. But that reminds me that we might need a list regarding all tools/portals we use and who is responsible/has permissions for what (github, dockerhub, netlify, ...) |
There was a problem hiding this comment.
I recall a long discussion we had when introducing the versioned docs. We left out latest (or switched to edge) on purpose. Rethinking this now imho it makes sense the way it is now as we also publish a latest tag together with the latest release. Only linking to specific docs site might break in future when sites gets removed or moved between sections. That is something that won't happen with strict versioned docs.
Looking at the edge docs we have a small tooltip that this is the latest documentation but not the latest stable one. Should we add a hint like this to the latest docs page as well stating that this is not the latest docs but the latest stable one?
Waiting for your opinion on that and if you both disagree with me i am happy to approve anyways.
Thanks @jrpear for putting this together and @polarathene i agree 100% on this "unrelated" comment:
I've been going through lots of the code, I appreciate how well commented it is and all of the links to relevant issues, I've come across a lot from @polarathene.
Hmm? Our docs are still versioned and you can link to them. Recent contributions from @jrpear has introduced a The plan was to default to directing users to those docs instead of I can't recall much about that earlier discussion, perhaps it related to I think our users are more likely to encounter the mismatch with image registries and the default |
AFAIK this is from refactoring that part of the docs during v12 by @georglauterbach I don't think you meant tooltip (popover / hover), but the admonition (coloured boxed content) we have on the main docs page? Once we release v12, the An easier fix is to remove releases after they're a certain age, simplifying the version selector to recent versions only. The markdown would still be accessible via git history on |
Let me give an example: You link to Now in v12 site-a gets moved to site-b Someone later in time using This even gets more complicated if not the site name switches but huge amount of the content is changed. Refs to this docs in old Issues or Posts will link to docs that might be completely out of context content wise. Do you get my point or am i overseeing something? |
Yes i meant that admonition
Thanks for clarifying, then i have no issue here. Just one further question. Will |
I understand that issue, but that isn't different to defaulting to
I think those were the answers you were after? If you're still concerned about Github Pages doesn't support redirects, but our preview service (Netlify) does. If we no longer used Github Pages (to publish, the Only drawback is the host would change from Github to Netlify, breaking all the existing links (unless we kept serving them with a banner about the new home). Obviously not ideal, but that gets worse the longer we stay on Github Pages (which we've used for approx 2 years now?). At the same time, I'd like to discard some older doc versions which'd have the same effect 😅 (on the plus side, with Netlify I have a 404 page that would at least better communicate to users about content being unavailable and direct them to the newer docs, GH pages last I checked didn't support 404 pages that well). |
wernerfred
left a comment
There was a problem hiding this comment.
Thanks for clarifying!
In fact i would like the option you mentioned with netlify the most (from a technical and UX perspective) but regarding the dockerhub discussion i would prefer to stick with as many onboard solutions as possible. Imagine netlify cancelling their OSS plans then we are totally screwed
|
Since today is Nyepi on Bali, and I already spent a lot of time sleeping and being in the sun (since I am not allowed to go outside), I can warrant 10mins on GH even though I'm on vacation 😂🙂 The PR looks good! I'd stay with GH Pages for our docs because it is unlikely we loose the ability to show our docs this way due to cancelled OSS plans. @jrpear we're always looking for contributors, moderators and in a best-case scenario, maintainers. If you are interested in DMS, I'd start by inviting you to the moderators team first. This will give you the ability to adjust issues and PRs. You'd also have access to the private discussions repo. Thereafter, I'd add you to the maintainers team :) Just tell me whether you're interested 🙂 |
|
Documentation preview for this PR is ready! 🎉 Built with commit: 9e1d7c6 |
|
PS: I will adjust the DockerHub README. |
Vercel or another competitor would likely offer the hosting. It'd be fairly cheap to host ourselves too, but there is plenty of fallback options if that was a concern 😅
The main advantage of moving off GH Pages would be the redirect feature. I am incredibly stretched for time right now and doubt anyone else would want to sort that migration out, so GH Pages it is 😝 I'll merge this then, and raise a TODO issue about updating earlier docs (or alternatively archiving them away at some point). Once again, thanks for the contribution! ❤️ |
edge docs to latest or relative linksedge version links to latest + fix links intended as relative not absolute
Yeah, I'd be happy to help moderate!
Great! Do you have permission to update the GitHub project description too? That still links to edge. |
You should have an invitation :)
Done :D |
Description
Change hard-coded links to point to the latest release rather than edge. Also change a few hard-coded links to be relative where I thought it made sense.
Finish addressing #3173
I don't know how the description on docker hub is maintained, maybe that needs a manual update. The github project description will certainly need a manual update.
Type of change