Make messages with required user action callouts (light blue boxes), i.e. 'How to find the admin area' and 'Contact Enterprise XYZ if you don't know why you are invited to be a manager'.
Extend the 'How to find the admin area' text to small screens but recommend using large screens at the same time.
All customer facing mails: Rephrase wording regarding contact email.
Order placement email: Replace 'Order confirmation' with 'Order summary' because the order is confirmed when the order cycle closes.
The rest are styling and wording improvements.
Authorization required mail (to shop): Add order summary table to email.
Authorize payment email (to customer): Add order summary table to email.
The order summary tables require the checkout_helper in the payment_mailer.rb.
Add 'Order summary' string to shared strings, because this will be needed in the order placement mailer for subscriptions as well (instead of 'Order confirmation').
The rest are styling and wording improvements.
Move order summary table to a shared folder to use it in other mailers as well, e.g. shipment mailer.
Don't see reason to use fixed width of 25 % for price column in order summary table.
Introduce class 'subsection' to allow for easier styling of shipping/pick-up information and special customer instructions (notes during checkout).
Introduce class 'detail' to allow styling of the payments list in emails without affecting the payments list in invoices.
Display section 'Payment method description' only if there is a description available.
Order confirmation email for shop: Add customer info headline and rearrange customer info
Order cancellation email for shop: Add distributor to intro. Add order summary table (I think this can be useful in case of cancellations.)
Invoice email: Add order number to intro
Order cancellation email: Rephrase wording
Signup confirmation email: Remove text 'Use send feedback button' (because it doesn't exist)
Reset password instructions email: Use link label instead of raw link (more common these days) and remove corresponding text 'Copy & paste the link'
Some subjects had a prefix (e.g. [Instance]), some subjects were misleading (e.g. failed payment email had 'Order Confirmation', some can be clearer by adding information like order number, distributor or coordinator
The 'invite manager' functionality was introduced in #2189. I searched for a decision to not use the 'set a password' text, but couldn't find one. So I assume it was forgotten. In my opinion it actually makes sense to have this information added.
To do so the shipment mailer and payment mailers need to know the @order
Remove the 'callout' class from distributor signoff for uniformity and to keep the callout available for call to actions only
Remaining translation keys for emails will be re-organized in a separate commit
Shared keys are used for greetings, signoffs, order confirmation string (incl. order number), contact distributor note and order edit note
Remaining translation keys for emails will be re-organized in a separate commit
Use wrap style full width background like header
Move 'Powered by...' message to footer and rephrase to suit all recipients (not shoppers only)
Always show link to openfoodnetwork.org instead of email (if present) because email will be available via icon in the row below now
Add social media buttons of instance to email footer
Make footer text and icons dark gray (as instance name in email header)
User the sites main logo instead of footer logo to allow for better contrast on gray background (dark text on gray instead of white text on gray)
Limit the max width and max height of the logo and resize the logo accordingly instead of forcing it to squeeze into a fixed size
Update specs
The key here is the enterprise_relationship. This is required for the supplier to have permission to see the orders.
Curiously, the unit test still passes. All will be revealed in the next commit..