How to change the date format in summary emails?

tophee · June 19, 2017, 7:03pm

I believe I have customized all text content elements containing dates but my summary emails don’t seem to be affected by that, they’re still going out with the “Month Day” format instead of “Day Month”. What am I missing? Where is the date format for digest emails set?

elijah · June 27, 2017, 4:23pm

I think you want this:

https://github.com/discourse/discourse/blob/master/app/views/email/_post.html.erb#L18

But I’m not 100% sure that it used for digests.

zogstrip · July 2, 2018, 4:15pm

github.com

discourse/discourse/blob/7eb5f783438bc0ea86fe22ffe6f2bb6c7d369d73/app/views/user_notifications/digest.html.erb#L115


      
          <table dir="<%= rtl? ? 'rtl' : 'ltr' %>" style="width:100%">
            <tbody>
              <tr>
                <td style="margin:0;padding:<%= rtl? ? '0 16px 0 0' : '0 0 0 16px' %>;text-align:<%= rtl? ? 'right' : 'left' %>;vertical-align:top;">
                  <p style="color:#8f8f8f;line-height:1.3;margin: 20px 0 0 0;">
                    <%= category_badge(t.category, inline_style: true, absolute_url: true) %>
                  </p>
                </td>
                <td style="margin:0;padding:<%= rtl? ? '0 0 0 16px' : '0 16px 0 0' %>;text-align:<%= rtl? ? 'left' : 'right' %>;vertical-align:top;">
                  <p class="text-right" style="color:#8f8f8f;line-height:1.3;margin:20px 0 0 0;font-weight:400;">
                    <%= short_date(t.created_at) %>
                  </p>
                </td>
              </tr>
            </tbody>
          </table>
          
          <table dir="<%= rtl? ? 'rtl' : 'ltr' %>" style="vertical-align:top;width:100%">
            <tbody>
              <tr>
                <td style="padding:<%= rtl? ? '0 16px 8px 8px' : '0 8px 8px 16px' %>; text-align:<%= rtl? ? 'right' : 'left' %>; width:100%;">

Which is defined in

github.com

discourse/discourse/blob/0dee603ffca1c733e9fdcfcb18a45471e09d4872/app/mailers/user_notifications.rb#L132-L138


      
          def short_date(dt)
            if dt.year == Time.now.year
              I18n.l(dt, format: :short_no_year)
            else
              I18n.l(dt, format: :date_only)
            end
          end

Which means you’ll have to change the short_no_year strings to change the date format in digests.

tophee · July 2, 2018, 5:34pm

Wow, thanks a lot! I missed that one because it doesn’t include “date” as a string. Are there any other “hidden” date or time format fields?

zogstrip · July 2, 2018, 5:36pm

I think most of them are here.

tophee · July 2, 2018, 5:47pm

Indeed, I found two more. But if those that you are linking to are “most of them” why is the list of date fields in admin/site-settings/customize/site_texts so much longer?Felt like dozens, but at least 20, I’d say.

zogstrip · July 2, 2018, 5:51pm

The others are the ones provided by rails or used in the web app (those starting with js.).

Topic		Replies	Views
Change of date in digest email body Feature activity-summary	10	1926	July 28, 2016
Date in digest emails are not localised Feature activity-summary	19	2712	April 3, 2015
What are some simple ways to improve digest emails Feature activity-summary	29	8915	January 5, 2017
Customize subject format for digest emails Feature activity-summary	3	1873	June 5, 2017
Summary Email Changes Feature email	24	5745	November 22, 2016

How to change the date format in summary emails?

Related topics