netlify / netlify-statuskit

Netlify StatusKit is a template to deploy your own Status pages on Netlify.
https://www.netlify.com/status-pages/
MIT License
238 stars 41 forks source link

Fix #25 #26

Closed steveoh closed 7 years ago

steveoh commented 7 years ago

Use .Site.RegularPages instead of .Site.Pages to fix #25.

Guard agains .Params being null with a default severity level.

TIL

Site.Pages

array of all content ordered by Date with the newest first. This array contains only the pages in the current language.

.Site.RegularPages

da shortcut to the regular page collection. .Site.RegularPages is equivalent to where .Site.Pages "Kind" "page".

verythorough commented 7 years ago

Tested and built!

Testing brought up an issue for me in that the hugo new incidents/whatever.md command doesn't use the bundled version of Hugo. I don't think it's a blocker for this PR, so I've filed issue #28 to raise the concern separately.

I'd love to hear your thoughts on the default severity as I mentioned above. Otherwise, LGTM!