Queue e-mails for background processing #124

Neues Issue

aheathershaw · 2019-07-13T18:23:31+01:00

aheathershaw hat

2019-07-13 18:23:31 +01:00

kommentiert

Besitzer

Queue e-mails in the database and use a Laravel queue worker to actually send the e-mails on a background job.

aheathershaw hat diesen Issue 2019-07-13 18:23:31 +01:00 zum Version 2.2.0-beta.1 Meilenstein hinzugefügt

aheathershaw hat sich das Issue 2019-07-13 18:23:31 +01:00 selbst zugewiesen

aheathershaw hat das Label

bug

2019-07-13 18:23:31 +01:00 hinzugefügt

aheathershaw hat den Titel von ~~Update forgot password form to BS4~~ zu Queue e-mails for background processing 2019-07-13 21:06:56 +01:00 geändert

aheathershaw hat das Label

wip

2019-07-13 21:07:28 +01:00 hinzugefügt

aheathershaw hat dieses Issue

2019-07-13 21:40:18 +01:00

aus einem Commit referenziert

#124, #125, #126: Started implementing e-mail queueing. Fixed the display of validation messages on the forgotten password form. Corrected the generation of the action URL when the APP_URL environment settings is not configured.

aheathershaw hat

2019-07-13 22:18:51 +01:00

kommentiert

Autor

Besitzer

The best way I can see this working is to use Laravel Notifications and dynamically adjust the “via” result.

(Laravel can not choose whether to queue a notification dynamically.)

We could have a custom notification channel that writes it to the database and a custom worker that processes that table.

This would however mean converting all Mailables to Notifications.

The best way I can see this working is to use Laravel Notifications and dynamically adjust the “via” result. (Laravel can not choose whether to queue a notification dynamically.) We could have a custom notification channel that writes it to the database and a custom worker that processes that table. This would however mean converting all Mailables to Notifications.

aheathershaw hat

2019-07-13 22:30:08 +01:00

kommentiert

Autor

Besitzer

Actually as of 5.3.7, you can use Mailables in conjunction with notifications:

https://laracasts.com/discuss/channels/laravel/laravel-53-using-mailable-inside-a-notification?page=0

Actually as of 5.3.7, you can use Mailables in conjunction with notifications: https://laracasts.com/discuss/channels/laravel/laravel-53-using-mailable-inside-a-notification?page=0

aheathershaw hat dieses Issue

2019-07-14 12:29:32 +01:00

aus einem Commit referenziert

#124: E-mails now send and log to the database as sent when queuing is not enabled

aheathershaw hat dieses Issue

2019-07-14 14:18:18 +01:00

aus einem Commit referenziert

#124: Updated the user self-registration required and user self-activated e-mails to be notifications so they can be queued.

aheathershaw hat dieses Issue

2019-07-14 14:35:30 +01:00

aus einem Commit referenziert

#124: Updated the change e-mail confirmation to be a notification

aheathershaw hat dieses Issue

2019-07-14 15:58:16 +01:00

aus einem Commit referenziert

#124: Converted the comment-related e-mails to be queueable

aheathershaw hat dieses Issue

2019-07-14 16:13:00 +01:00

aus einem Commit referenziert

#124: Made the queue settings configurable through the .env file

aheathershaw hat das Label

wip

2019-07-14 16:15:03 +01:00 entfernt

aheathershaw hat diesen Issue

2019-07-14 16:15:07 +01:00

geschlossen

aheathershaw hat

2019-07-14 16:16:09 +01:00

kommentiert

Autor

Besitzer

This is now complete. To run the queue, use the command below:

php artisan bt-queue:send-emails

or reference the example systemd service in resources/systemd/blue-twilight-email.service.

This is now complete. To run the queue, use the command below: `php artisan bt-queue:send-emails` or reference the example systemd service in `resources/systemd/blue-twilight-email.service`.

aheathershaw hat dieses Issue

2019-07-14 16:16:38 +01:00

aus einem Commit referenziert

#124: Corrected the base class in the comment replied to mailable.

aheathershaw hat

enhancement

hinzugefügt, und

bug

2019-09-07 11:07:55 +01:00 entfernt

Anmelden, um an der Diskussion teilzunehmen.