Skip to content

display dates on posts #9

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 14 commits into from
Nov 14, 2018
Merged

Conversation

ihardy
Copy link
Contributor

@ihardy ihardy commented Jul 3, 2018

Changes:

  • applied post layout to _posts directory to add display date
  • moved posts for static pages into the _pages directory and kept the page layout so post date is not displayed on those pages
  • include the _pages directory in config.yml
  • added the date from the title of the static page posts to the header yml in case it's needed in the future

I'm new to jekyll and this is more changes than I thought it would be to show the date, so if changing the templates doesn't fit w/your plans for the site I won't be if you want to reject this.

Fixes #2

@ihardy ihardy force-pushed the t2-display-post-dates branch from c99fb59 to d08f861 Compare August 13, 2018 12:54
@kaitlinnewson
Copy link
Member

This looks good to me - any other thoughts @wickr, or can we go ahead and merge?

@wickr
Copy link
Contributor

wickr commented Nov 14, 2018

Thanks for the reminder to look at this again. At first this looked like a lot of changes but they seem to be straightforward.

@wickr wickr merged commit 7f8e29e into code4lib:master Nov 14, 2018
@wickr
Copy link
Contributor

wickr commented Nov 14, 2018

@ihardy Thanks for working on this. Your changes are up on the live site.

kaitlinnewson added a commit that referenced this pull request Nov 14, 2018
@ihardy ihardy deleted the t2-display-post-dates branch November 22, 2018 00:23
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.

3 participants