I'd like to add a year selector to the heatmap and maybe more things. Since that will reveal even more information about users we might want to hide the heatmap for those who didn't log in. A proper check would be the terms of use acceptance, but that's not possible until #5706 is merged. You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5829 -- Commit Summary -- * Test if max_id is present in heatmap data * Split heatmap rendering tests * Show heatmap only to logged in users -- File Changes -- M app/controllers/users_controller.rb (38) M test/controllers/users_controller_test.rb (62) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/5829.patch https://github.com/openstreetmap/openstreetmap-website/pull/5829.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5829 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________ rails-dev mailing list rails-dev@openstreetmap.org https://lists.openstreetmap.org/listinfo/rails-dev
