-
Notifications
You must be signed in to change notification settings - Fork 37
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
🔨 Collections Category/Group fixes #170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of slug changes that need human intervention. Looks good otherwise
_schedule/talks/2022-10-18-09-45-t0-come-on-in-the-waters-fine-making-python.md
Outdated
Show resolved
Hide resolved
layout: session-details | ||
schedule_layout: full | ||
permalink: /social-event/board-game-night/ | ||
permalink: /social-events/board-game-night/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to check if the board game night post links here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't see it referenced in the post.
_schedule/talks/2022-10-19-09-45-t0-keynote-state-of-the-object-relational.md
Outdated
Show resolved
Hide resolved
@drewbrew good catch. I'm won't merge until we sort it out. This has bitten us in the past, and I see a one-liner that I think will fix it going forward. I'll push and fix. |
Will follow up with a better perma-fix, but short term the process.py script won't regenerate permalinks once they are set. |
Closes #169.
This is a workaround for Jekyll Collections (no clue when this changed) using category == subfolder instead of what we assign it. I updated
process.py
to validate/fix/sync and re-ran it over our schedule data.