[openstreetmap/openstreetmap-website] Redaction title can be blank. (#2033)
Andy Allan
notifications at github.com
Wed Jan 31 15:43:51 UTC 2024
> We want ensure the presence of Title and Description both,
Yes please
> For missing titles and description we need fill them up with the id of the redaction?
I would suggest "Redaction #{id}" (i.e. the word "Redaction" followed by the id number) as the title, and "No description" as the description. But it's not super important since I don't think there are many (any?) on the osm.org servers. It might affect other deployments though.
> In a single migration we have to backfill our data and change the column null to false right?
> Since we are using strong migrations do we need to change null first and then Validate it inside a seperate migration as stated by the strong migration documentation?
Yes, whichever approach works best with strong migrations. See https://github.com/ankane/strong_migrations?tab=readme-ov-file#setting-not-null-on-an-existing-column
Thanks for looking into this!
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/2033#issuecomment-1919371487
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/issues/2033/1919371487 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240131/0784c87b/attachment.htm>
More information about the rails-dev
mailing list