Scheduled actions : Send a mail to all EME registered people

By /

Events Made Easy › Forums › Bug fixed or feature request implemented › Scheduled actions : Send a mail to all EME registered people

Viewing 14 posts - 1 through 14 (of 14 total)
  • Author
    Posts
  • Mon 22 Jun 2020 at 16:30 #61374
    Anonymous
    Inactive

    Hi Franky
    There’s something about the way it works that I can’t understand. In the function “Send a mail to all EME registered people for upcoming events that will happen in the next”, whatever the period indicated and the frequency the answer is always “Pending emails will be sent in batches of 0 Once every …”, and no sending is planned.
    And therefore no mail sending seems to be scheduled and none appears in the mailing tab.

    Thank you for your feedback. Have a nice day.
    Frédéric.

    Mon 22 Jun 2020 at 18:09 #61375
    Franky
    Keymaster

    The setting “Send out queued mails in batches of” mentions the interval and amount. That should be correctly configured too. I’ll add an extra check that the amount of mails to be send in a specific interval can’t be zero.

    Mon 22 Jun 2020 at 18:15 #61376
    Anonymous
    Inactive

    Yes it is fifty Once every 5 minutes. What I’m wondering about is the number “0” in the return screen after “Apply”

    Mon 22 Jun 2020 at 18:57 #61377
    Franky
    Keymaster

    O yes indeed, big bug. I wonder why nobody reported that earlier, this is probably in there for months already.
    This fixes it:
    https://plugins.trac.wordpress.org/changeset/2328734/

    Mon 22 Jun 2020 at 19:20 #61379
    Anonymous
    Inactive

    I think there is still a bug. When I click on “Apply”, no message is queued and the screen return is the same as if I had clicked on the “apply” button from above (Queued mails will be send out in batches of…)

    Mon 22 Jun 2020 at 21:24 #61380
    Franky
    Keymaster

    The message was not fixed, that’s correct. But what do you mean with “no message is queued”?

    Edit: sorry, I fixed it. Apparently another typo was in there for months as well …
    This is the latest fix: https://plugins.trac.wordpress.org/changeset/2328790

    And maybe you should first apply this fix too, to fix the message when pressing Apply:
    https://plugins.trac.wordpress.org/changeset/2328781

    Mon 22 Jun 2020 at 22:34 #61382
    Anonymous
    Inactive

    Ok. I have downloaded the two files. All seems ok. The mails are queued and sent correctly.

    Mon 22 Jun 2020 at 22:53 #61383
    Anonymous
    Inactive

    I guess emails are only sent if the setting Massmail is yes ?

    Mon 22 Jun 2020 at 23:13 #61384
    Franky
    Keymaster

    Indeed, for the newsletter that is the case.

    Mon 22 Jun 2020 at 23:36 #61385
    Anonymous
    Inactive

    Thanks for helping out on such short notice. Your plugin is so rich that I haven’t tested everything yet 😉

    Mon 22 Jun 2020 at 23:44 #61386
    Franky
    Keymaster

    Well, you helped me find another bug, so I should be thanking you too 🙂

    Tue 23 Jun 2020 at 14:51 #61390
    Anonymous
    Inactive

    Hi Franky
    It seems to me that the massmail parameter is not taken into account when sending the newsletter. All my users receive it, including those where the parameter is “no”.

    Tue 23 Jun 2020 at 16:18 #61391
    Franky
    Keymaster

    You’re correct, I accidentally reversed logic testing there. I now changed it to be more easy to read and test. This is the change:
    https://plugins.trac.wordpress.org/changeset/2329377/

    Tue 23 Jun 2020 at 17:47 #61392
    Anonymous
    Inactive

    Thanks, that’s ok now.

  • Author
    Posts
Viewing 14 posts - 1 through 14 (of 14 total)
  • The forum ‘Bug fixed or feature request implemented’ is closed to new topics and replies.
Scroll to Top