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 date filter for publikationen #123

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

Fix date filter for publikationen #123

wants to merge 1 commit into from

Conversation

k-nut
Copy link
Member

@k-nut k-nut commented Nov 18, 2024

The current list of publications is broken since the hugo update.

The reason for this is that in publikationen.html layout, we use GroupByDate.
In the upgrade, we have renamed a lot of fields to publishedDate. In our config, we set

[frontmatter]
date  = [":filename", ":default"]

instructing hugo to first look at the filename when trying to find a date, and then use the default list. This list notably doesn't include any field called publishedDate, only publishDate is listed.
This led to the filters only taking into account the dates from the file names and no longer from the front matter. Since only three fields had valid dates, only those would be considered for the date filter.

This change hence renames all the dates field in the publikationen pages to date again to be consistent with how hugo usually does things.

@k-nut k-nut requested a review from stefanw November 18, 2024 14:13
Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for silver-smakager-d8f15e ready!

Name Link
🔨 Latest commit 91aa1cb
🔍 Latest deploy log https://app.netlify.com/sites/silver-smakager-d8f15e/deploys/673b4b62633da10008f7d84d
😎 Deploy Preview https://deploy-preview-123--silver-smakager-d8f15e.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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