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

Clean up config.toml #14

Closed
2 tasks done
AngeloStavrow opened this issue Sep 14, 2018 · 2 comments
Closed
2 tasks done

Clean up config.toml #14

AngeloStavrow opened this issue Sep 14, 2018 · 2 comments
Assignees
Labels
bug report Something isn't working

Comments

@AngeloStavrow
Copy link
Owner

AngeloStavrow commented Sep 14, 2018

Right now there's a bit too much redundancy between config.toml and the page front-matter, like author/AuthorName parameters. I removed the .Site.Params.AuthorName parameter in a9cb556, but we should better define site and post authors.

To do:

  • Default to .Params.Author but fallback to .Site.Author (don't forget footer.html!)
  • Remove .Site.Params.CopyrightOwner for .Site.Copyright
@AngeloStavrow AngeloStavrow self-assigned this Sep 14, 2018
@AngeloStavrow AngeloStavrow added the bug report Something isn't working label Sep 14, 2018
@AngeloStavrow AngeloStavrow added this to the v0.1 public release milestone Sep 14, 2018
@AngeloStavrow
Copy link
Owner Author

Oddly, I can't get .Site.Author as configured in the example Hugo site's config.toml file to render.

@AngeloStavrow
Copy link
Owner Author

We can work around this with a .Site.Params.Author in the config.toml file; we'll have to specify this in the README docs. Additionally, since we're using variable overwrites, we'll need to specify a minimum version for Hugo of 0.48.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant