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

Andy Allan notifications at github.com
Tue Nov 20 12:25:06 UTC 2018


I should also say that I don't like the scope names suggested in this PR, since I'm concerned that they might be misleading. For example:
 * `trackable_ordered` - this scope also consists of identifiable traces, not just trackable traces. However, I realise that the existing "Trace.trackable?" method is a bit of a mind-bender since it creates a second meaning of "trackable" already.
* `non_trackable_unordered` - this scope has strictly ordered results for privacy reasons, they are not 'unordered' in the usual meaning of the word.


-- 
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-440255443
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181120/9658ea45/attachment.html>


More information about the rails-dev mailing list