Problem/Motivation

Some minor improvements, see below.

Proposed resolution

  1. Adjust the padding to match the original mail more closely.
  2. Only use the store address if it is actually set. Otherwise fallback to Mailer Policy, which gives a nicer experience because it sets a display name.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 symfony_mailer.commerce.3372634-3.patch1.18 KBadamps
#2 symfony_mailer.commerce.3372634-2.patch1.7 KBadamps

Comments

AdamPS created an issue. See original summary.

adamps’s picture

adamps commented
Status: Active » Needs review
StatusFileSize
new symfony_mailer.commerce.3372634-2.patch1.7 KB
adamps’s picture

adamps commented
StatusFileSize
new symfony_mailer.commerce.3372634-3.patch1.18 KB

Oops patch #2 included a file from a separate issue

  • AdamPS committed f8625fc6 on 1.x 
    Issue #3372634 by AdamPS: Improvements to commerce emails
adamps’s picture

adamps commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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