[openstreetmap/openstreetmap-website] Move tracepoint ordering logic into tracepoint model, and add tests (#2068)

krubokrubo notifications at github.com
Tue Nov 20 02:57:12 UTC 2018


That makes sense to me.  I moved the tracepoint privacy ordering into scopes on the model now, doing the union in the controller.  I'm just chaining them with the existing bbox scope, which seems to me like the simplest way.  I had some mixups with my commits (you can see my force-push history here), but cleared up the rubocop offenses and other issues I could find.  Let me know if it's suitable?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2068#issuecomment-440119527
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181119/55b89598/attachment-0001.html>


More information about the rails-dev mailing list