Daily Digest bug

[dcook]

Hi @baris

Sorry, I am not clear on how to set the digest title, are you saying I should add those 3 lines to the email.json file you referenced?

Thanks
Duane

[baris]

No, those 3 lines should already be in the email.json file.

[julian]

@dcook, did you by any chance customize your email digest template in the ACP? You should revert any changes (or set it back to the default) from Settings -> Email. I think it is referencing an old language tag digest.title, which we do not use any longer.

[dcook]

Hi, I am not seeing any Digest settings in Settings / Email other than this:

[julian]

Hm, what version of core and persona theme are you running?

[dcook]

You are running NodeBB v1.13.0

nodebb-theme-persona
Persona theme for NodeBB
Installed 10.1.27 | Latest 10.1.27

[julian]

Right... in that settings/email page, there should be a big black text box for you to customize the template.

Use the dropdown to change the template to the digest template, and then revert it back to the default template... I'm thinking there might be a custom template set.

[magnusvhendin]

So, we have some modifications to the visual style of the template. Since emails can be a bit tricky to work with I just changed the template tag to [[email:digest.title.{interval}]]. But instead of showing up as digest.title in the emails sent, it now shows up as digest.title.alltime.

I only tried to send a test email so maybe that's the issue. Should I report this as an issue? I guess email:digest.title.alltime should be added to the appropriate language file.

[julian]

No, I think that is actually okay. In the emails we send out, we pass in day, week, or month depending on the digest. For the test email, the value alltime is sent in, purely so the digest is populated with something.

We just don't happen to have a language tag for alltime

[magnusvhendin]

Great, then there shouldn't be any issues when the digests are sent for real.