Change of date in digest email body

AI-generated summary

The discussion revolves around changing the date format in the digest email body to a localized format. Mischief points out that the current format, “mm-dd-year”, can be confusing for users based in the UK. gerhard mentions that they submitted a fix for the issue, but it hasn’t been merged yet.

After some discussion, techAPJ makes a temporary change to use a universal date format, “August 23rd”, until gerhard’s fix is merged. eviltrout later merges gerhard’s pull request.

However, Mischief reports that the issue has reverted back after an update. codinghorror asks techAPJ to investigate, and techAPJ makes another change to use a localized date format with year.

Continuing the discussion from Date in digest emails are not localised:

There is still a date localisation problem in the actual digest message body where at the start of the digest it says "A brief summary of “[%{site_name}] since your last visit on mm-dd-year”.

This date format can be very confusing for our members who are mainly based in the UK.

If it could be localised or alternatively a full textual representation of the month used it would be appreciated.

1 Like

That date is currently not translatable in Transifex. I submitted a fix for it some time ago…

https://github.com/discourse/discourse/pull/3787

3 Likes

I think there were problems with that commit as I recall, which is why it hasn’t been merged.

@techapj can we change the digest default text so it uses a more universal date format? Maybe something like

Since your last visit on August 23rd

where the year is optional if it was in the current calendar yar.

1 Like

Well, the date format should take the user’s locale into account, anything else is just a hack.

My PR worked perfectly fine in my dev environment. I haven’t been told about any problems, but if there are some, I’ll try to fix them.

3 Likes

Done via:

https://github.com/discourse/discourse/commit/734c272de81baac8a70801e257a994a8a6e9f616


Will switch to localized date again once @gerhard’s PR is merged.

4 Likes

I’ve merged in the PR from @gerhard now!

3 Likes

A huge thank you for making this change. It is a great improvement! …on this occasion the like button just didn’t seem enough!

4 Likes

This seems to have reverted back. I’m once again getting digests which are saying: A brief summary of XXXXXX Forum since your last visit on mm-dd-yyyy

Seems like it happened during an update between the 26th of December and the 2nd of January.

2 Likes

Hmm I think you are right, @techapj can you check into this please?

3 Likes

Okay, I made the date (with year) in digest email universal

https://github.com/discourse/discourse/commit/90d41a994acb8b24356fd9b86d9daf01a26bfbef

(and localized, thanks @gerhard!)

https://github.com/discourse/discourse/commit/26bba1f48139ead65ccd4a81a88b640e5bc0e906

4 Likes