[openstreetmap/openstreetmap-website] Add hide/show selection for contribution heatmap on edit profile page (PR #5853)

[Andy Allan]

@gravitystorm commented on this pull request.



> @@ -0,0 +1,5 @@
+class AddShowContributionHeatmapToUsers < ActiveRecord::Migration[7.2]
+  def change
+    add_column :users, :show_contribution_heatmap, :boolean, :default => true, :null => false

> * it ok to create a column with a default value in a large table?

We use [strong migrations](https://github.com/ankane/strong_migrations) to check that the migrations are safe to run in production. This means that contributors don't need to learn all the edge cases, and maintainers don't need to know them all either and can merge PRs with confidence if CI is green.

Previously strong migrations would flag this situation for PG < 11, but it now supports only PG 12+ and so it [dropped the check last year](https://github.com/ankane/strong_migrations/commit/1e428665978185bc3b21a36db723dd91c10b5e12#diff-2c318d74d04a82865bea4780a048a37f4871393db99d20b798eb19516c188104R44-L46)

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

Message ID: <openstreetmap/openstreetmap-website/pull/5853/review/3593480304 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251218/8850029b/attachment.htm>