Skip to content

Update post-list.hbs added slash to page/2#61

Merged
cathysarisky merged 3 commits intoTryGhost:mainfrom
jvanderstelt:main
Mar 5, 2026
Merged

Update post-list.hbs added slash to page/2#61
cathysarisky merged 3 commits intoTryGhost:mainfrom
jvanderstelt:main

Conversation

@jvanderstelt
Copy link
Copy Markdown
Contributor

@jvanderstelt jvanderstelt commented Sep 19, 2024

Because ghost automaticaly adds slashes to urls page/2 should have a slash at the end otherwise it is causing an extra redirect.


Note

Low Risk
Single template hyperlink change plus a newline-only edit; minimal behavioral impact beyond reducing redirects.

Overview
Updates the post list template to link to {{@site.url}}/page/2/ (trailing slash) for the “See all” pagination CTA, avoiding an extra redirect on Ghost installs configured with slash-terminated URLs.

Also normalizes the file ending by ensuring a final newline.

Written by Cursor Bugbot for commit 63363ab. This will update automatically on new commits. Configure here.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 5, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 95b8e2b4-996d-4931-bb7a-c506045e6ee7

📥 Commits

Reviewing files that changed from the base of the PR and between 22c2d73 and 63363ab.

📒 Files selected for processing (1)
  • partials/components/post-list.hbs

Walkthrough

A pagination link in the post-list template component was updated to include a trailing slash in its href attribute, changing from /page/2 to /page/2/. The anchor text "See all" with arrow icon remains unchanged. An end-of-file newline was added as a minor formatting adjustment. No exported or public API declarations were modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change—adding a trailing slash to the /page/2 URL in post-list.hbs to prevent extra redirects.
Description check ✅ Passed The description is directly related to the changeset, explaining the rationale for adding the trailing slash to fix Ghost's automatic URL slash handling.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

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

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Free Tier Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Bugbot Autofix is kicking off a free cloud agent to fix this issue. This run is complimentary, but you can enable autofix for all future PRs in the Cursor dashboard.

Comment thread partials/components/post-list.hbs Outdated
Comment thread partials/components/post-list.hbs Outdated
@cathysarisky
Copy link
Copy Markdown
Member

Thanks! This is a good fix that should have happened quite a while ago. Merging now!

@cathysarisky cathysarisky merged commit aab69b0 into TryGhost:main Mar 5, 2026
1 check passed
adenau pushed a commit to adenau/technodabbler-source-theme that referenced this pull request Mar 29, 2026
* Update post-list.hbs added slash to page/2

---------

Co-authored-by: Cathy Sarisky <42299862+cathysarisky@users.noreply.github.com>
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.

2 participants