@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev