Skip to content
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

Add Order status update in Order View page #16008

Merged
merged 6 commits into from Oct 18, 2019

Conversation

@sarjon
Copy link
Contributor

sarjon commented Oct 18, 2019

Questions Answers
Branch? develop
Description? Add Order status update in Order View page
Type? improvement
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? Part of #15818
How to test? Not yet.

This change is Reviewable

@sarjon sarjon requested a review from PrestaShop/prestashop-core-developers as a code owner Oct 18, 2019
@@ -107,3 +107,13 @@ admin_orders_change_currency:
_controller: PrestaShopBundle:Admin/Sell/Order/Order:changeCurrency
requirements:
orderId: \d+

admin_orders_resend_email:
path: /{orderId}/history/{orderHistoryId}/statuses/{orderStatusId}/resend-email

This comment has been minimized.

Copy link
@sarjon

sarjon Oct 18, 2019

Author Contributor

@matks wdyt about this path?

This comment has been minimized.

Copy link
@matks

matks Oct 18, 2019

Contributor

Makes sense 👍 from the path we can guess the target, so it works

@sarjon sarjon force-pushed the sarjon:m/orders/change-status branch from 48c86a7 to 0839fab Oct 18, 2019
@matks
matks approved these changes Oct 18, 2019
@matks
matks approved these changes Oct 18, 2019
@matks matks added this to the 1.7.7.0 milestone Oct 18, 2019
@matks

This comment has been minimized.

Copy link
Contributor

matks commented Oct 18, 2019

Merged to ease cooperation, the QA and product reviews will happen on next PR to be built on top of this one

@matks matks merged commit 13aab5a into PrestaShop:develop Oct 18, 2019
2 checks passed
2 checks passed
PrettyCI Code formatting
Details
Travis CI - Pull Request Build Passed
Details
@sarjon sarjon deleted the sarjon:m/orders/change-status branch Oct 18, 2019
@matks matks added the migration label Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.