Skip to content

Conversation

@carrythebanner
Copy link
Collaborator

2nd round of merch sales end on June 11th; don't merge this change before June 12.

@ionous
Copy link
Contributor

ionous commented Jun 11, 2025

nice! i like the config variable.
if you want a shortcut, instead of:
{{ if eq (.Param "festival.merchSaleActive") true }}{{ partial "cal/pp-merch.html" . }}{{ end }}

this should also work:
{{ if .Site.Params.festival.merchSaleActive }}{{ partial "cal/pp-merch.html" . }}{{ end }}

@carrythebanner
Copy link
Collaborator Author

Thanks @ionous! Definitely prefer that simpler syntax, I'll update (and use it in a couple of other similar checks).

@carrythebanner carrythebanner marked this pull request as ready for review June 12, 2025 21:46
@carrythebanner carrythebanner merged commit e9cfcef into shift-org:main Jun 12, 2025
4 checks passed
@carrythebanner carrythebanner deleted the merch-sale-config branch June 12, 2025 21:47
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.

2 participants