The HTML order template has some CSS issues that need to be fixed :

  • Inline the logo and "thanks for your order"
  • Same blue color for order number and "contact us" than the "view your order" button
  • The address should be displayed the same way for all lines
  • Add spacing on top and bottom of the items table
  • 1 line per subtotal and price under the table
CommentFileSizeAuthor
#3 Selection_029.png49.49 KBroysegall
Current HTML order template56.94 KBguguss

Comments

guguss’s picture

guguss commented
Assigned: cookiz » amitaibu
roysegall’s picture

roysegall commented
Assigned: amitaibu » roysegall
roysegall’s picture

roysegall commented
Status: Active » Needs review
StatusFileSize
new Selection_029.png49.49 KB

I applied the fixes. The gerrit page: https://code.drupalcommerce.org/141.

Also, attached a screen shot.

bojanz’s picture

bojanz commented
Status: Needs review » Needs work

Changing status per Augustin's code review in gerrit.

guguss’s picture

guguss commented

Need some quick fixes :

  • More spacing between "view your order" and "Ship to"
  • More spacing between the address and "items in your order"
  • items in your order => Items in your order
  • ship to => Ship to
guguss’s picture

guguss commented
Status: Needs work » Fixed

Commit has been merged.

guguss’s picture

guguss commented
Assigned: roysegall » theo_
Status: Fixed » Active

It seems that no email is sent to the non-admin user after an order is proceed !

@theo : can you check why ?

guguss’s picture

guguss commented
Priority: Normal » Major
amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
Assigned: theo_ » itamar

Re-assign.

itamar’s picture

itamar commented
Assigned: itamar » theo_

Order-completed email does get sent to normal authenticated users when I try it locally. I'm unable to reproduce the issue.

theo_’s picture

theo_ commented
Priority: Major » Normal
Status: Active » Needs review

The problem was that, we were trying to send a mail to the order's owner before assigning the owner to the order ...

Juste updated the weight of the rule rules_commerce_order_message_order_notification to happen after user creation.

https://code.drupalcommerce.org/212

bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Merged, good job.

Status: Fixed » Closed (fixed)

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