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 revert and deactivate methods to allow actions to take place when… #7705

Merged
merged 2 commits into from Dec 1, 2016

Conversation

Projects
None yet
3 participants
@wildtangent
Contributor

wildtangent commented Nov 29, 2016

… a promotion is removed from the cart

@mauazua

This comment has been minimized.

Show comment
Hide comment
@mauazua

mauazua Nov 30, 2016

Contributor

looks ok, please rebase your branch with master :)

Contributor

mauazua commented Nov 30, 2016

looks ok, please rebase your branch with master :)

Joe Connor added some commits Nov 29, 2016

Joe Connor
@wildtangent

This comment has been minimized.

Show comment
Hide comment
@wildtangent

wildtangent Nov 30, 2016

Contributor

@mauazua any idea why my tests are failing? can't see any actual error in CircleCI so its not an actual failing spec

Contributor

wildtangent commented Nov 30, 2016

@mauazua any idea why my tests are failing? can't see any actual error in CircleCI so its not an actual failing spec

@mauazua mauazua merged commit c3340fa into spree:master Dec 1, 2016

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
hound No violations found. Woof!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment