Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(page-list): enforce intended behavior on GitHub-pages hosted websites #1685

Open
wants to merge 1 commit into
base: v4
Choose a base branch
from

Conversation

dechantoine
Copy link

@dechantoine dechantoine commented Dec 27, 2024

Hello! 👋

I've been scratching my head for a while trying to figure out why URLs on my Quartz GitHub Pages site were occasionally missing the repo name:

Desired URL: https://username.github.io/repo-name/a/b/c
Obtained URL: https://username.github.io/a/b/c
This discrepancy often led to frustrating 404 errors.

After some digging, I came across this issue and its corresponding fix. While the original fix addressed the issue for specific cases, I realized I was encountering the same problem but with the PageList component. 😄

This PR proposes a similar solution: adding a trailing slash to folder URLs to ensure pathResolution works correctly, by prevent landing on URLs that are missing a slash.

Thank you for your time and consideration! I'm happy to provide more details if needed. 😊

Best regards,
Antoine

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