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

Add ROOT_URL to table of content #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

NilsIrl
Copy link

@NilsIrl NilsIrl commented Jun 10, 2020

Fix #15

This is the upstreaming of aadibajpai/blog@afe0210

@pastly
Copy link
Owner

pastly commented Jun 10, 2020

I'm busy the next few days, but do want to find the time to at lest spot check that this works I understand the full reason for the problem and that this addresses it.

If I haven't said anything more by June 17th, please ping me again.

@NilsIrl
Copy link
Author

NilsIrl commented Jun 10, 2020

No worries, take your time, here's this patch in production. In the previews, the table on content used the assume the blog was a the root / not /blog (that's not the case anymore).

Co-authored-by: Aadi Bajpai <me@aadibajpai.me>
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.

[BUG] ROOT_URL variable is not respected in hyperlinks from home page
2 participants