fix: resolve 37 broken links detected by lychee CI checker#14562
Merged
skjnldsv merged 1 commit intofeat/ci-lycheefrom Apr 24, 2026
Merged
fix: resolve 37 broken links detected by lychee CI checker#14562skjnldsv merged 1 commit intofeat/ci-lycheefrom
skjnldsv merged 1 commit intofeat/ci-lycheefrom
Conversation
Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/ae18fa2c-5300-4ccd-9db7-6f2c00da3a4b Co-authored-by: nextcloud-command <88102737+nextcloud-command@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
nextcloud-command
April 24, 2026 07:44
View session
| such as Amazon S3, SMB/CIFS file servers and FTP servers… | ||
| in Nextcloud. Your Nextcloud server administrator controls which of these are | ||
| available to you. Please see `Configuring External Storage (GUI) | ||
| <../../admin_manual/configuration_files/external_storage_configuration_gui.rst>`_ in the Nextcloud Administrator's |
Member
There was a problem hiding this comment.
are you sure this is working/a good idea?
As far as I recall the RST links got resolved to HTML for HTML and also worked in the PDF by scrolling to the related section.
Not they are relative links in the PDF that lead to a relative HTML page that does not exist on the server / local PC?
But for cross manual like here we actually need to use absolute links
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The lychee link checker added in #14231 found 37 broken links because the commit converting absolute
https://docs.nextcloud.com/server/latest/…URLs to relative paths used.rstextensions — but lychee checks the built HTML stage, where only.htmlfiles exist.Changes
.rst→.htmlin all relative cross-manual hyperlinks across admin, developer, and user manualsmysql_4byte_support.rst— reverted self-link back to absolute URL targeting v20 docs (the#mariadb-10-2-or-earliersection no longer exists in current docs)coding_standards/index.rst— fixed relative path depth for codereviews link (../→../../)user_manual/desktop/uninstallation.rst— fixed 4 links pointing to non-existentadmin_manual/desktop/configfile.html; the file lives inuser_manual/desktop/, so changed to relativeconfigfile.html#configuration-fileuser_manual/files/access_webdav.rst— fixed self-referencing linkaccess_webdav.rst#anchor→#anchorwebdav.rst/office/configuration.rst— wrapped barehttps://andhttp://text in inline code literals (https://) to prevent docutils from auto-linking them as relative broken paths.github/workflows/sphinxbuild.yml— added--exclude-path '.*/_static/.*'to lychee config to skip template files (e.g.openapi.html) that reference CDN resources not present in the stage🖼️ Screenshots