I realized today while thinking about the "place order on another date" feature of order creation that our order receipt email doesn't actually include the date the order was placed or payment was made. This is a problem in general, not just for late or delayed placement of admin orders, as the timestamp of an email being sent or received may not correlate to the time of the order or payment.

For reference, here's the order receipt email as sent by a clean installation of Commerce Kickstart:

I think we have three options we need to choose from:

  1. Add the order placed date to the email.
  2. Add the payment placed date to the email in the payment section.
  3. Both of the above.

I tend to think option #1 is our safest bet for core. Aside from the reality that Order doesn't depend on Payment, which we could make accommodation for like we did to get shipping details in there, we also don't know if all payment methods on a site would even support or properly be represented as being placed on a given date (e.g., bank transfers with multi-day approvals, checks, invoices, etc.).

Thus, I'd propose something like:

This changes our current header a bit similar to other receipts I see from retailers I visit that email copies of receipts. Another option would be to put it in the footer near the order total, though I think it deserves greater prominence.

Issue fork commerce-3461898

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

rszrama created an issue. See original summary.

jsacksick made their first commit to this issue’s fork.

jsacksick’s picture

StatusFileSize
new26.03 KB

MR opened, let's see which tests are failing. The receipt looks like this now :)

jsacksick’s picture

Issue summary: View changes
StatusFileSize
new26.67 KB

Added divs and a 10px top margin to the order date:

Also pushed a commit to hopefully address the test failures.

jsacksick’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new26.7 KB

With 5px marging, and hopefully pushed another fix addressing the remaining test failures...

rszrama’s picture

Status: Needs review » Reviewed & tested by the community

5px works for me. Matches what we're doing elsewhere in the template.

  • jsacksick committed 48e2eb69 on 8.x-2.x
    Issue #3461898 by jsacksick, rszrama: Add the order date to the order...

  • jsacksick committed 4b585437 on 3.0.x
    Issue #3461898 by jsacksick, rszrama: Add the order date to the order...
jsacksick’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.