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

Send signals on order and order line status change. #2047

Closed
wants to merge 4 commits into from

Conversation

michaelkuty
Copy link
Contributor

@michaelkuty michaelkuty commented May 18, 2016

Make possible to handle status change for order and their lines.
Needs documentation, but I can make it if you say that these signals are useful and will be merged.

@codecov-io
Copy link

codecov-io commented Aug 1, 2016

Current coverage is 83.30% (diff: 100%)

Diff Coverage File Path
•••••••••• 100% src/oscar/apps/order/signals.py
•••••••••• 100% src/oscar/templatetags/currency_filters.py
•••••••••• 100% src/oscar/apps/order/abstract_models.py

No coverage report found for master at 156a4b8.

Powered by Codecov. Last update 156a4b8...ab8ca99

@sasha0
Copy link
Member

sasha0 commented Aug 2, 2016

I think this PR now contains changes, unrelated to the signals introduction.

@michaelkuty
Copy link
Contributor Author

Sorry my bad, new PR #2075

@michaelkuty michaelkuty closed this Aug 2, 2016
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