-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Admin][Order] Add order notes #5319
Conversation
Arminek
commented
Jun 22, 2016
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
License | MIT |
@Arminek There is a field |
@Arminek We could rename it to notes indeed. |
<div class="ui attached segment" id="sylius-order-notes"> | ||
{{ order.notes }} | ||
</div> | ||
{% endif %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing blank line
b9900bc
to
ea324af
Compare
6318c0e
to
76aae77
Compare
@@ -134,6 +142,7 @@ protected function getDefinedElements() | |||
'items_table' => '#items table', | |||
'product_row' => 'tbody tr:contains("%name%")', | |||
'order_total' => 'td:contains("Total")', | |||
'extra_notes' =>'#sylius_checkout_summary_notes', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just notes
? Add I would keep alphabetical order ;)
@@ -43,6 +43,7 @@ public function __construct(CustomerRepositoryInterface $customerRepository, Fac | |||
|
|||
/** | |||
* @Transform :customer | |||
* @Transform /^made by "[^"]+"$/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should only getCustomerByEmail
and never create a customer. Add second method to this context.
/** | ||
* @Given there is an administrator identified by :email | ||
*/ | ||
public function theStoreHasCustomerServiceAccountIdentifiedBy($email) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change customer service to administrator in shared storage and method name too.