Skip to content

Fixing docs SSL link - #218

Open
WesCook wants to merge 1 commit into
pelican:mainfrom
WesCook:patch-1
Open

Fixing docs SSL link#218
WesCook wants to merge 1 commit into
pelican:mainfrom
WesCook:patch-1

Conversation

@WesCook

@WesCook WesCook commented Sep 10, 2026

Copy link
Copy Markdown

The Webserver Configuration page links to /docs/panel/guides/ssl, not the correct /docs/guides/ssl/.

The link path was right in code, but relative links may not be processed correctly. Absolute paths do work as seen on other doc pages (eg. docker.mdx), so I've used that instead.

Summary by CodeRabbit

  • Documentation
    • Updated the “Creating SSL Certificates” link to use the correct documentation path.

The link path was right, but relative links may not be processed correctly.  Absolute paths do work as seen on other doc pages (eg. docker.mdx).
@netlify

netlify Bot commented Sep 10, 2026

Copy link
Copy Markdown

Deploy Preview for pelica ready!

Name Link
🔨 Latest commit 8794a61
🔍 Latest deploy log https://app.netlify.com/projects/pelica/deploys/6aa2f5265a8e630007502ea0
😎 Deploy Preview https://deploy-preview-218--pelica.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 project configuration.

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 754b20a0-0597-48de-93dd-cb247486da16

📥 Commits

Reviewing files that changed from the base of the PR and between 6d24d3a and 8794a61.

📒 Files selected for processing (1)
  • docs/panel/webserver-config.mdx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details
🔇 Additional comments (1)
docs/panel/webserver-config.mdx (1)

9-9: 🎯 Functional Correctness

Keep the existing link. Docusaurus and the repository configuration use /docs/guides/ssl without a trailing slash.


📝 Walkthrough

Walkthrough

The SSL certificate documentation link in the web server configuration guide now uses the absolute /docs/guides/ssl path.

Changes

SSL Documentation

Layer / File(s) Summary
Update SSL documentation link
docs/panel/webserver-config.mdx
The SSL certificate link now uses /docs/guides/ssl instead of ../guides/ssl.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: notareyouscared

Merge Risk: 🔵 Low · up to 8794a

The link now targets the correct SSL guide, but its missing trailing slash could cause a redirect or broken link if the documentation router requires the canonical URL. This is low risk but should be confirmed.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: fixing the SSL link in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit hops along the docs path
SSL links now avoid the wrong turn
From relative trails to roots they dash
Clear directions guide each fern
The guide is fixed with a twitching nose

Comment @coderabbitai help to get the list of available commands.

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.

1 participant