Skip to content

N31: Init #948

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

Merged
merged 4 commits into from
Feb 13, 2022
Merged

N31: Init #948

merged 4 commits into from
Feb 13, 2022

Conversation

17cupsofcoffee
Copy link
Collaborator

Part of #912

Copy link
Member

@ozkriff ozkriff left a comment

Choose a reason for hiding this comment

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

lgtm except for the same comment

maybe we should start specifying the date filed in the drafts: iirc, zola doesn't render them otherwise

I guess the date field could be added to the template date = TODO?

@17cupsofcoffee
Copy link
Collaborator Author

@ozkriff: I can try that, but I get the feeling Zola will error if the date isn't formatted right - if it doesn't work, I'll just set it to 2022-03-01.

@ozkriff
Copy link
Member

ozkriff commented Feb 13, 2022

I get the feeling Zola will error if the date isn't formatted right

yeah, it surely will. I meant adding date = TODO to newsletter-template.md

@17cupsofcoffee
Copy link
Collaborator Author

Aaaah, that makes a lot more sense - done :)

@17cupsofcoffee 17cupsofcoffee merged commit f458d30 into source Feb 13, 2022
@17cupsofcoffee 17cupsofcoffee deleted the 17cupsofcoffee-n31-init branch February 13, 2022 14:07
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