[openstreetmap/openstreetmap-website] Fix month label index desync causing missing labels (PR #6725)
Tom Hughes
notifications at github.com
Tue Jan 20 13:31:00 UTC 2026
tomhughes left a comment (openstreetmap/openstreetmap-website#6725)
I didn't completely figure things out last night but I think this is the root cause of the problem:
https://github.com/openstreetmap/openstreetmap-website/blob/5b3210c5f637596ed65e75c2173990055b11fe5c/app/helpers/heatmap_helper.rb#L17
The intention is that if you start with say February then each time you get to January you add 12 so that the numbers carry on going up but in this case we're starting in January as the first value and unnecessarily add 12 so the values start from 13 as the first "month".
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6725#issuecomment-3772903405
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6725/c3772903405 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260120/93debcba/attachment.htm>
More information about the rails-dev
mailing list