gohugoio / hugoBasicExample

MIT License
281 stars 281 forks source link

Assign multiple content types #37

Closed onedrawingperday closed 5 years ago

onedrawingperday commented 5 years ago

As I found out since #33 was commtited in Hugo type cannot be an array but only a string.

Therefore this commit reverts type back into a string.

So for the HugoBasicExample to provide support for both post AND posts content types, I have assigned the type posts to half of the content files, while the other half will retain the existing post type.

This will fix the issue of several themes that use the type posts in their list pages and since https://github.com/gohugoio/hugoThemes/pull/547 currently have empty list pages, like the following:

Hamburg Dusky Neon Potato Potato Darl Hyde Hyde

Also this PR is relevant to the following theme submission: https://github.com/gohugoio/hugoThemes/issues/545#issuecomment-466672234 because when this is merged the HugoBasicExample will provide support for both post AND posts content types.

cc: @digitalcraftsman

digitalcraftsman commented 5 years ago

Is this PR ready to be merged? I was about to add a comment about the conflict that Git detected. But this has been resolved a moment ago.

onedrawingperday commented 5 years ago

@digitalcraftsman

36 is now cleaned up and ready to be merged first.

Then please merge this PR.

But for both of these to take effect on the website a manual deploy needs to be triggered on Netlify.