@hlfan commented on this pull request.


> @@ -76,6 +76,7 @@
     required(:api_version).filled(:str?)
     required(:max_request_area).filled(:number?)
     required(:max_note_request_area).filled(:number?)
+    required(:traces_enabled).filled(:bool?)

In #7138, we had a similar situation where we landed on the double negative 
`unless hide_signup` to communicate the default more clearly.
I think this is the way to go here too.

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/7156/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to