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

Change sylius_update_shipment state machine callback. #2045

Merged

Conversation

liverbool
Copy link
Contributor

Q A
Bug Fix? yes
New Feature? n
BC Breaks? n
Deprecations? n
Tests Pass? n
Fixed Tickets #2044
License MIT
Doc PR

do: [@sm.callback.cascade_transition, 'apply']
args: ['object.getShipments()', 'event', '"prepare"', '"sylius_shipment"']
do: [@sylius.callback.shipment_states, 'updateOrderShipmentStates']
args: ['object', '"prepare"']
Copy link
Contributor

Choose a reason for hiding this comment

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

@liverbool I prefer not to put the 2nd argument.

@kayue
Copy link
Contributor

kayue commented Oct 22, 2014

It was quick 👍

@liverbool
Copy link
Contributor Author

@kayue i have fulltime :)

@liverbool liverbool force-pushed the confirm-order-statemachine-callback branch from 87e1453 to 996d790 Compare October 22, 2014 12:33
pjedrzejewski pushed a commit that referenced this pull request Oct 22, 2014
…-callback

Change `sylius_update_shipment`  state machine callback.
@pjedrzejewski pjedrzejewski merged commit 6235aa1 into Sylius:master Oct 22, 2014
@pjedrzejewski
Copy link
Member

Thank you @liverbool! 👍

@liverbool liverbool deleted the confirm-order-statemachine-callback branch February 24, 2015 03:48
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.

None yet

3 participants