[openstreetmap/openstreetmap-website] Refactor Heatmap: Manual Labels & Yearly Template (PR #5906)

Tom Hughes notifications at github.com
Fri May 9 16:35:28 UTC 2025


tomhughes left a comment (openstreetmap/openstreetmap-website#5906)

I honestly don't know where to start with this but it seems to me that it takes us from using a third party component with some reasonably simple configuration to writing a vast amount of code to take over much of the work which that component was doing.

I don't even understand how it works... Previously we had a domain of months broken down into days, which seems reasonable, but now we have a domain of days broken down into years? Except clearly we don't if look at the chart it produces... Is the new custom `yearly` domain badly named or something?

The question I keep asking myself every time I look at this and try and understand it is, is all this worth it just to make the heatmap look a bit closer to the github one? At least I assume that's the goal...

It would certainly be nice if we could get it to stop at the current date instead of the end of the current month, which seems to be the main noticeable effect of this to me when I try it, but is it worth this amount of work to get that?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5906#issuecomment-2867181800
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5906/c2867181800 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250509/49e2686e/attachment.htm>


More information about the rails-dev mailing list