@HolgerJeromin commented on this pull request.
> + domain: [0, heatmapData.reduce((max, item) => {
+ return item.total_changes > max ? item.total_changes : max;
+ }, 0)]
Faster for 365 entries? I am not sure.
Definitely not worth the [ugly
readability](https://medium.com/better-programming/think-again-before-you-use-array-reduce-28f785b5aea9).
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5833#discussion_r2007256347
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5833/review/2705389...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
