Closed pgiraud closed 7 years ago
Hello, thanks a lot! That was actually a mistake on my part - didn't realize that pushing to the branch again will update the PR. Will review.
Dne 6. 1. 2017 16:04 napsal uživatel "Pierre GIRAUD" < notifications@github.com>:
@Piskvor https://github.com/Piskvor I noticed your last commit in this pull request.
926 https://github.com/hotosm/osm-tasking-manager2/pull/926
I think however that it deserves a different pull request. Thus the current one. Also I improved it a bit so that all the buttons (ie. mark as done, validate and invalidate as well) are taken into account.
Please review.
You can view, comment on, or merge this pull request online at:
https://github.com/hotosm/osm-tasking-manager2/pull/932 Commit Summary
- indicate activity, prevent multiple clicks
File Changes
- M osmtm/static/js/project.js https://github.com/hotosm/osm-tasking-manager2/pull/932/files#diff-0 (6)
Patch Links:
- https://github.com/hotosm/osm-tasking-manager2/pull/932.patch
- https://github.com/hotosm/osm-tasking-manager2/pull/932.diff
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/hotosm/osm-tasking-manager2/pull/932, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3-96_duVLqYyb3YtpzXdwAlVbAWp8xks5rPlf4gaJpZM4LcyjG .
@Piskvor I noticed your last commit in this pull request. https://github.com/hotosm/osm-tasking-manager2/pull/926
I think however that it deserves a different pull request. Thus the current one. Also I improved it a bit so that all the buttons (ie. mark as done, validate and invalidate as well) are taken into account.
Please review.