[openstreetmap/openstreetmap-website] Add user profile heatmap visualization for contributions (PR #5402)

Anton Khorev notifications at github.com
Tue Jan 21 03:43:06 UTC 2025


@AntonKhorev commented on this pull request.



> @@ -53,6 +53,23 @@ def show
     if @user &&
        (@user.visible? || current_user&.administrator?)
       @title = @user.display_name
+
+      one_year_ago = 1.year.ago

Is this done to freeze the starting point because we're not sure when the block below runs? Then why is the same not done with the ending point? And shouldn't it be truncated to a day?

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

Message ID: <openstreetmap/openstreetmap-website/pull/5402/review/2563416158 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250120/a280dd53/attachment.htm>


More information about the rails-dev mailing list