[openstreetmap/openstreetmap-website] [WIP] Simplify GPS trace visibility levels (PR #7146)

Ruben L. Mendoza notifications at github.com
Fri Jun 12 20:19:17 UTC 2026


@Rub21 commented on this pull request.



>  
-    if visibility
-      visibility.v
-    elsif current_user.preferences.find_by(:k => "gps.trace.public", :v => "default").nil?
-      "private"
-    else
-      "public"
-    end
+    "trackable"

Good idea. I moved it to User#default_trace_visibility, following the default_diary_language pattern.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7146/review/4488926644 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260612/4f4c9a44/attachment.htm>


More information about the rails-dev mailing list