Open DaveParr opened 4 years ago
Related to this: I think files like data/schedule/Talk01.toml
should be in the example site directory and not included in the theme. As it currently is, it's not possible to display the organizers without also listing these phantom speakers. Even if you deleted data/schedule/
from the top-level site, it still inherits these files from the theme. Including these files directly in the theme was another change made during Hacktoberfest: https://github.com/satRdays/hugo-satrdays-theme/commit/c3b64e44142ab1d4796265703cb9799135f92be9#diff-150e59ddb24eb64e62a4f1307fa077d2de18ee46f0cc33b7d4fa30056da9553b
Please let me know if there is some other way to exclude these files.
For now I overrode the default team.html
with a version that deleted the speakers section
https://github.com/satRdays/Columbus2021/commit/095803ef2cd156ca0c4126d33f3fa54cf0d7317a
@jdblischak I'm really torn about implementing this. I'd really appreciate a second opinion. I have some pros and cons which I'm trying to debate.
This probably isn't a complete list, but I'm effectively trying to weigh up the burden for people contributing and maintaining the project, who might have a better idea about the hugo best-practices and interals anyway, vs making things as simple as possible at the deployment and customising stages.
Any thoughts?
I'm frustrated that the (top level repo)[https://github.com/satRdays/satRday_site_template] needs to have files like
config.toml
updated when changes to this repo are made. I also want to be able to have a way to run a command likehugo serve
to preview the changes I make to the repo run in this repos root, not have to move up a level to preview them.The reason for the previous design was to make it easier to make usable repos quickly from the template, without the individual event organisers having to worry too much about making a sub-repo. As it is they need to worry about that anyway when picking up updates, so IMHO my previous plan kind of doesn't work anyway.
Any soultion needs to add to this repo AND also update the top level repo to account for this change.