[openstreetmap/openstreetmap-website] Send trace import result notification immediately (#2313)
Tom Hughes
notifications at github.com
Mon Jul 15 20:47:33 UTC 2019
If we delay sending failure notifications then they will fail because the trace will no longer exist.
As trace imports are running in the background anyway there doesn't seem to be any good reason to defer the emails.
Fixes #2312
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/2313
-- Commit Summary --
* Send trace import result notification immediately
-- File Changes --
M app/jobs/trace_importer_job.rb (6)
M test/jobs/trace_importer_job_test.rb (18)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/2313.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2313.diff
--
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/2313
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190715/e45018f5/attachment.html>
More information about the rails-dev
mailing list