climateaction-tech / branch-theme

The theme used in the branch magazine by climateAction.tech
13 stars 2 forks source link

Fix bug with "very high" carbon intensity #41

Closed jacklenox closed 10 months ago

jacklenox commented 10 months ago

Our code and design can't currently handle an additional fourth tier of carbon intensity: "very high", and as such this change is currently breaking the site.

This quick fix tells our carbon intensity checker to treat "very high" simply as "high".

Merged and working on staging: https://branch-staging.climateaction.tech/

mrchrisadams commented 10 months ago

For work on the green web foundation platform, we've started to adopt a ship, show, ask approach for changes, as outlined below, to speed up the getting small changes like this into production.

https://martinfowler.com/articles/ship-show-ask.html

On this basis, I'm accepting the change as I know it fixes a known bug, and the consequences of merging in bad code are pretty low anyway.