New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tests for tracepoint ordering #2059
Comments
krubokrubo
added a commit
to krubokrubo/openstreetmap-website
that referenced
this issue
Nov 19, 2018
I submitted a pull request for this. I kept the logic exactly as it was, although I'm still a bit doubtful about the meaning of the different visibility values (private/public/trackable/identifiable). Let me know if it's suitable? |
krubokrubo
added a commit
to krubokrubo/openstreetmap-website
that referenced
this issue
Nov 20, 2018
The controller still does the union of trackable ordered points with non-trackable unordered points, getting each from the model. Closes openstreetmap#2059
krubokrubo
added a commit
to krubokrubo/openstreetmap-website
that referenced
this issue
Nov 20, 2018
The controller still does the union of trackable ordered points with non-trackable unordered points, getting each from the model. Closes openstreetmap#2059
krubokrubo
added a commit
to krubokrubo/openstreetmap-website
that referenced
this issue
Nov 20, 2018
The controller still does the union of trackable ordered points with non-trackable unordered points, getting each from the model. Closes openstreetmap#2059
krubokrubo
added a commit
to krubokrubo/openstreetmap-website
that referenced
this issue
Nov 21, 2018
The controller uses these scopes to do the union of trackable (and identifiable) points with anonymized private (and public) points. Closes openstreetmap#2059
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
cdb42d2 was a hotfix for an important privacy problem, but hasn't yet received any tests.
I would also recommend refactoring the complex logic out of the controller and into the TracePoint model, which will make it easier to test too.
The text was updated successfully, but these errors were encountered: