We are a medium size community and are using Discourse very happily.
2 days ago (16-Nov-2015) I updated to the latest version and I think that since then email notifications to subscribers of categories stopped working for everyone.
Digest emails still work properly and test emails do too.
What can be done? Where should I look? Which information can I add to help understand the issue?
Can you repro this @techapj? This is the third time I have seen someone mention it so something might be up with watching categories.
I also have the same issue with the email notifications not being sent to subscribers or original posters when they get a reply, along with the translations issue from last beta build (v1.5.0.beta5 +17).
I get an error 500 while trying to preview the email (it works when the language is set to english, but error 500 in french):
In logs the only issue there is is that:
If that can help this is what I can see in sidekiq:
Maybe everything is related to translations ?
Makes sense, @gerhard this is kind of a traumatic change for our users (at least in non-English locales) so I hope it was worth it…
Something feels buggy somewhere to me… @riking added the fallback logic a while back. Somehow it regressed.
We are using the site in Hebrew.
In site/logs I get an error about 2-3 times a minute:
"Job exception: translation missing: he.time.formats.short_no_year"
In the details of the error (attached), the user notifications are mentioned.Job exception error.txt (7.6 KB)
I can’t see earlier errors or when it started since all the last 1000 errors are these.
In site/sidekiq there seem to be many fails since Nov 16 (About 50% out of the processed). I don’t know exactly what to look for in there. Do you want the link to look at it yourselves or direct me what to look for?
Also, yesterday I again updated to the latest version (there were just a few commits), and since then I get on the dashboard an error about translations - “translation missing: he.dashboard.failing_emails_warning”.
Could problems with translation cause email failures? Or does it just mean that I don’t receive the warning about the failed emails?
(I should note that we don’t update the translations on transifex regularly enough).
Let me know what other information would be helpful.
Thanks a lot for your support!
Ok, I read @heptoilabas 's posts again so I understood what to get from sidekiq:
I’m quite sure this used to work. Actually, I fixed the exact same issue a few months back.
Anyway, I’ll look into it tonight unless someone else beats me to it.
This could be related to @eviltrout local override work… if it gets tricky you can skip it … its a work in progress.
What do you suggest I do in the meanwhile? Can I downgrade without loosing data from the past week?
I’m working on it now. I should have a fix ready soon!
I figured it out. It’s complicated to explain how I made this mistake, but monkey patching something in Rails and declaring a class look exactly the same. I thought some code @riking wrote was a class definition when it was actually a patch. So I missed some fallback logic. Confusingly it worked in most cases despite missing much of the logic!
It’s fixed in this commit, should be in tests-passed shortly:
edit: Since it was important I also back-ported the fix to beta. If you update via admin you should get it there too.
You guys are absolutely awesome!
Thanks a lot for your great support and of course for Discourse itself
That’s because all the missing logic was covering failure cases