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

[Documentation] Concepts: Shipments #5848

Merged
merged 1 commit into from
Aug 30, 2016

Conversation

CoderMaggie
Copy link
Member

Q A
Doc fix? yes
New docs? no
BC breaks? no
Related tickets partially #5275
License MIT

@CoderMaggie
Copy link
Member Author

Waiting for a helping hand with the calculators here. :)

request_shipping:
from: [cart]
to: ready
cancel:
Copy link
Member

Choose a reason for hiding this comment

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

It is something for later, but I guess we are missing a transition between partially_shipped and cancelled

@CoderMaggie CoderMaggie force-pushed the docs/shipments branch 2 times, most recently from 697d4d9 to abb83ce Compare August 30, 2016 07:16
...
A **Shipment** is a representation of a shipping request for an Order. Sylius supports multiple shipments per one Order.

How to create a Shipment programmatically?
Copy link
Member

Choose a reason for hiding this comment

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

I'd remove this part, as it is not that simple, it works, but the structure is not correct. We should use the processor to create shipments for an order, but we can document it later.

@CoderMaggie CoderMaggie changed the title [WIP] [Documentation] Concepts: Shipments [Documentation] Concepts: Shipments Aug 30, 2016
@pjedrzejewski pjedrzejewski merged commit 7b2b9b3 into Sylius:master Aug 30, 2016
@pjedrzejewski
Copy link
Member

Thank you Magda!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants