[openstreetmap/openstreetmap-website] add bulk_upload (#1995)
Xuyang Jia
notifications at github.com
Tue Sep 18 01:55:01 UTC 2018
> Well my point was why is there nothing in the PR to explain what it's about? or even in the commit messages...
>
> Above all else, if it's just optimisations why is it a new method rather then just improving the existing method?
Sorry for making you confused, I will write some descriptions later...
The new bulk_upload method is slightly different from old upload method on response in some case, but won't break original logic. If it's OK, I will make a commit to replace the old one. Thanks!
--
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/1995#issuecomment-422227372
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180917/b324031a/attachment-0001.html>
More information about the rails-dev
mailing list