Open jarrodmillman opened 6 months ago
@jarrodmillman Please see below my current plan for this issue and let me know if I've missed anything. Will be starting on the items presently. Thanks.
Plan: How to proceed.
[ ] In scientific-python-hugo-theme
: The theme repo itself.
yamlToTables
shortcode with a tomlToTables
one: Add tomlToTable by stefanv · Pull Request #548 · scientific-python/scientific-python-hugo-theme · GitHub
tomlToTables
shortcode.tomlToTables
.yamlToTables
.[ ] On other sites: Sites that use scientific-python-hugo-theme
.
scientific-python-hugo-theme
that includes tomlToTable
.yamlToTable
with tomlToTable
.Questions: What else do I need to know?
yamlToTable
or yamltotable
? Same for the TOML version.References: Relevant resources.
Adding tomlToTable
shortcode (draft): Add tomlToTable by stefanv · Pull Request #548 · scientific-python/scientific-python-hugo-theme · GitHub
Changing card shortcodes to use TOML: Use toml for card data by jarrodmillman · Pull Request #468 · scientific-python/scientific-python-hugo-theme · GitHub
yamlToTable
shortcode added (2021): Add yamlToTable shortcode by stefanv · Pull Request #7 · scientific-python/scientific-python-hugo-theme · GitHub
I can take care of tomltotable and removing yamltotable. It would be worth reviewing if there are other places that would benefit from using toml, but it isn't a priority. I will keep this open as more of a meta-issue for now.
While working on the grid of cards (i.e., embedded shortcodes), we ran into issue using yaml (see #468)
We should replace yamltotables (see #7 and #11). We should probably make a release with tomltotables, convert all the sites to use it, and remove yamltotables in the next release.
We may also want to use more toml in shortcodes as we make more complicated shortcodes.