Hi, thank you for your plugin I saw some issues that I'll list later but for now I have a question about the tool.
In the source code I saw that you're directly sending the mail with the Mail::send method.
But I'm scared about scalability: I'll have to send around 12k email for an important user base. and I don't think that the server will love that.
I don't want to get a partial sending issue. (memory error, timeout etc.)
Did you try to use Queues instead ? https://octobercms.com/docs/services/mail#queueing-mail
Thank you for your time. Adrion
it may be simply an extra option in message model, this way no breaking change for current users.
Hi,
Thanks, for using our plugin.
The server definitely will not like the process. We have never been reported about such a case and to be honest, we have never used the plugin with such a userbase.
We can add this feature in a new version if you like but it can be more like the third week of February. Are you okay with it?
Best regards,
Tomasz Strojny
I need to manually modify the helper, it will be a shortcut for this time :P
But yes, I'll be glad if you can add it later Thank you !
Regards, Adrion
Hi Adrion,
Hope your shortcut works well :)
The problem is that I have promised you that I'll take care of it this week. Unfortunately, my calendar is still colorful and if it's not a high priority for you I would like to take care of it in my spare time. Probably in a few weeks.
Is it acceptable for you?
Thanks,
Tomasz Strojny
Hi Tomasz,
Yes the manual queue worked as expected :) !
I totally understand, my current fix will do the trick for now, if you can do it before the end of April it will be awesome. I don't want to let a custom workaround in your plugin, it will be easier to maintain for the next team that will handle my project.
Many Thanks !
Adrion
Hi,
Queues work in 1.0.14. The only thing you have to do is override the default config for the plugin.
Create a file config/initbiz/newsletter/config.php
with the following content:
<?php
return [
'useQueue' => true
];
Best regards,
Tomasz Strojny
1-7 of 7