Hi.

This module seems to do exactly what I want to do in my current project, thanks! and reviewing the code and I found this issue:

Right now the personal_digest_cron() method is not working, because it has the wrong queueWorker name:

$queue = \Drupal::queue('personal_digest');

Instead of 'digest_mail' it should be: digest_mail otherwise the items aren't going to be processed by the DigestMail QueueWorker and the mails won't be sent.

CommentFileSizeAuthor
#2 2830080-wrong-queuename.patch632 bytesgnuget
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gnuget created an issue. See original summary.

gnuget’s picture

FileSize
632 bytes
gnuget’s picture

Status: Active » Needs review
matslats’s picture

Status: Needs review » Fixed

Thanks a lot.
When you are satisfied with it let me know and I'll push beta 1.
Note that this module is not at all tested.

matslats’s picture

Status: Fixed » Closed (fixed)