add AbstractOrder.return_order #187
Conversation
radekholy24
force-pushed
the
return_order
branch
from
5203751
to
b1f8672
Compare
radekholy24
force-pushed
the
return_order
branch
2 times, most recently
from
c4e1d61
to
468bdf5
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #187 +/- ##
==========================================
- Coverage 86.79% 86.73% -0.06%
==========================================
Files 64 64
Lines 2347 2375 +28
==========================================
+ Hits 2037 2060 +23
- Misses 310 315 +5 ☔ View full report in Codecov by Sentry. |
CHANGELOG
Outdated
| @@ -1,6 +1,10 @@ | |||
| django-plans changelog | |||
| ====================== | |||
|
|
|||
| 1.1.0 (unreleased) | |||
| ------------------ | |||
| * Add `AbstractOrder.return_order` | |||
There was a problem hiding this comment.
AbstractOrder.return_order() to be clear that it is a function.
docs/source/index.rst
Outdated
| @@ -47,6 +47,7 @@ Contributors: | |||
| * Victor Safronovich <[email protected]> | |||
| * Dominik Kozaczko <http://dominik.kozaczko.info> | |||
| * Petr Dlouhý <[email protected]> | |||
| * Blender Kit <[email protected]> | |||
There was a problem hiding this comment.
BlenderKit <[email protected]> and/or give yourself a credit
| @@ -13,6 +13,7 @@ | |||
| framework. | |||
|
|
|||
| """ | |||
|
|
|||
There was a problem hiding this comment.
Probably unnecessary modification of this file.
There was a problem hiding this comment.
Yeah, I think so too, but it won't pass Black otherwise... 🤦
radekholy24
force-pushed
the
return_order
branch
from
468bdf5
to
318e260
Compare
Symmetrically to
AbstractOrder.complete_order, we want to return an order and reduce the account.