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

[Admin][Order] Order features #4898

Merged
merged 11 commits into from
May 12, 2016
Merged

Conversation

GSadee
Copy link
Member

@GSadee GSadee commented Apr 28, 2016

Q A
Bug fix? no
New feature? no
BC breaks? no
Related tickets
License MIT
  • Browsing all orders
  • Seeing basic information about an order
  • Seeing an order with items and total
  • Seeing shipping fees of an order
  • Seeing taxes of an order
  • Seeing discounts of an order
  • Deleting an order

@GSadee GSadee changed the title [Admin][Order] Order features [WIP][Admin][Order] Order features Apr 28, 2016
@pjedrzejewski pjedrzejewski added the Admin AdminBundle related issues and PRs. label Apr 28, 2016
@GSadee GSadee force-pushed the admin-orders branch 6 times, most recently from 1b93b02 to dfaa54a Compare May 4, 2016 21:46
@GSadee GSadee force-pushed the admin-orders branch 2 times, most recently from 0161c1a to 4045e74 Compare May 6, 2016 13:52
And I am logged in as an administrator

@ui
Scenario: Seeing an order basics information
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seeing basic information about the order

Then the product named "Angel T-Shirt" should be in the items list
And the order's items total should be "€39.00"
And the order's shipping total should be "€0.00"
And there should be a shipping charge "Free €0.00"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be swapped with previous step (as in the scenario below).

@GSadee GSadee force-pushed the admin-orders branch 4 times, most recently from 90225e2 to 796145c Compare May 12, 2016 06:49
@pjedrzejewski pjedrzejewski merged commit bc3f948 into Sylius:master May 12, 2016
@pjedrzejewski
Copy link
Member

Nice work, thank you Grzesiu! 👍

@GSadee GSadee deleted the admin-orders branch September 22, 2017 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants