[openstreetmap/openstreetmap-website] [WIP] Use deliver_later for email sending (#2038)

Andy Allan notifications at github.com
Wed Oct 31 15:51:40 UTC 2018


This PR builds on #2037 and configures email sending as a background task.

Note that this depends on having the setup for delayed_job (#2037) already merged, along with having workers running in production, before this can be deployed - so I'm marking it as WIP until those prerequisites are met.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/2038

-- Commit Summary --

  * Set up Delayed Job as the backend for Active Job
  * Use deliver_later for all email sending

-- File Changes --

    M .rubocop_todo.yml (4)
    M Gemfile (1)
    M Gemfile.lock (6)
    M app/controllers/changeset_controller.rb (2)
    M app/controllers/diary_entry_controller.rb (2)
    M app/controllers/messages_controller.rb (2)
    M app/controllers/notes_controller.rb (2)
    M app/controllers/users_controller.rb (10)
    M config/application.rb (3)
    M config/environments/test.rb (3)
    A db/migrate/20181031113522_create_delayed_jobs.rb (22)
    M db/structure.sql (62)
    M test/controllers/changeset_controller_test.rb (12)
    M test/controllers/diary_entry_controller_test.rb (24)
    M test/controllers/messages_controller_test.rb (46)
    M test/controllers/notes_controller_test.rb (12)
    M test/controllers/users_controller_test.rb (82)
    M test/integration/user_creation_test.rb (584)
    M test/test_helper.rb (1)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/2038.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2038.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/2038
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181031/1d2c7e7e/attachment-0001.html>


More information about the rails-dev mailing list