theforeman / foreman-documentation

Documentation for the Foreman Project and its ecosystem
https://docs.theforeman.org
Creative Commons Attribution Share Alike 4.0 International
22 stars 95 forks source link

Fix broken links in Installing Proxy #3327

Closed asteflova closed 1 month ago

asteflova commented 2 months ago

What changes are you introducing?

Fixing several broken links found in Installing Proxy.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

We received a list of broken links in https://issues.redhat.com/browse/SAT-27794. About half of these actually needed fixing, the other ~half was just references to guides for a future d/s release.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

EDIT: Apart from just fixing the broken links, I also changed some of them into xrefs in cases when the target was in the same guide.

Checklists

Please cherry-pick my commits into:

github-actions[bot] commented 2 months ago

The PR preview for a815ae09da8e4b46a12ee79f7731d7dfa74c77d4 is available at theforeman-foreman-documentation-preview-pr-3327.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

asteflova commented 2 months ago

Tech review not necessary because this is just about fixing links > tech review done

We can ignore the Vale check: lowercase dns is part of a section ID.

maximiliankolb commented 1 month ago

@asteflova Please rebase to "master" and resolve the merge conflict.

asteflova commented 1 month ago

Merged to "master" and cherry-picked:

c9e8b85d6c..bd9d5c75db 3.12 -> 3.12