-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
slowing down my test suite by factor 15 #46
Comments
well, this is the quick hotfix. don't use the layout to send mails and just mock the make_bootstrap_mail and let it do default. i wonder - did nobody ever had this issue before or do people not test their emails? |
I'm experiencing similar issue when sending a newsletter with Sidekiq, 50 emails get processed quickly then queue is stuck to 1 or 2 minutes and then it starts working again and so on. |
@majksner i never figured out why. the compilation of the bootstrap is insanely slow, but once you have it, it should be done and persist 🤷 |
This might not be related to this gem. I'm using mailgun in production but when I use |
I also just tested it (via CLI). It takes >3seconds for compiling. How it take so long to transpile one html format to another? For me that is ok, I just compile it before deploying. But this is still quite extreme. Especially in development, where I recompile on every changes and the browser preview has a very noticeable delay. |
Performance is important, it is something we are constantly trying to improve. After more of the new alpha 1.0.0 version is solidified performance will become a bigger priority 👍 |
well, that escalated quickly.
i wonder how we can speed that up for our test suite? send out emails without bootstrap? can't really be an option for a test suite somehow.
The text was updated successfully, but these errors were encountered: