[openstreetmap/openstreetmap-website] Add query parameter that to suppress display of signup UI elements (PR #7138)

Tom Hughes notifications at github.com
Sun Jun 7 10:49:10 UTC 2026


@tomhughes commented on this pull request.



> +    signup_param = ref_params["allow_signup"].first
+    @allow_signup = !signup_param || signup_param != "false"

As `nil != "false"` is true I think you can just write this as:

```suggestion
    @allow_signup = ref_params["allow_signup"].first != "false"
```

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

Message ID: <openstreetmap/openstreetmap-website/pull/7138/review/4444905367 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260607/32957e99/attachment.htm>


More information about the rails-dev mailing list