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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add static file's basename to its url_placeholder #7908

Merged
merged 3 commits into from Apr 13, 2020

Conversation

ashmaroli
Copy link
Member

  • This is a 馃悰 bug fix.

Summary

Jekyll 4.0 introduced a minor change (via #7315) in how URLs for static files were generated whereby the logic for static files in a collection became more responsive to the associated collection's url_template. So when a collection's url_template is customized at client-side to utilize :name, static files should adapt accordingly.

Context

Resolves #7904

@DirtyF DirtyF added this to the 4.1 milestone Nov 25, 2019
@ashmaroli ashmaroli marked this pull request as ready for review December 25, 2019 16:01
@ashmaroli ashmaroli requested a review from a team December 25, 2019 16:02
Copy link
Member

@DirtyF DirtyF left a comment

Choose a reason for hiding this comment

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

Thanks for adding a test 馃檹

@DirtyF
Copy link
Member

DirtyF commented Apr 13, 2020

@jekyll: merge +fix

@jekyllbot jekyllbot merged commit be70c3e into jekyll:master Apr 13, 2020
@jekyllbot jekyllbot added the bug label Apr 13, 2020
jekyllbot added a commit that referenced this pull request Apr 13, 2020
@ashmaroli ashmaroli deleted the staticfile-basename-url branch April 26, 2020 09:14
ashmaroli added a commit to ashmaroli/jekyll that referenced this pull request Apr 26, 2020
Add static file's basename to its url_placeholder
This backports be70c3e to 4.0-stable
ashmaroli added a commit that referenced this pull request Apr 26, 2020
Add static file's basename to its url_placeholder
This backports be70c3e to 4.0-stable
@jekyll jekyll locked and limited conversation to collaborators Apr 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Static file in collection (with permalink set) causes build to fail
3 participants