Skip to content

Commit

Permalink
Add TODO list item
Browse files Browse the repository at this point in the history
Co-authored-by: Chris Todorov <[email protected]>
Co-authored-by: Adam Mueller <[email protected]>
Co-authored-by: Alistair Norman <[email protected]>
Co-authored-by: benjamin wil <[email protected]>
Co-authored-by: Harmony Bouvier <[email protected]>
Co-authored-by: Sofia Besenski <[email protected]>
  • Loading branch information
7 people committed Jan 17, 2025
1 parent abebe44 commit dbc5940
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ In-Memory Order Updater TODO
- [x] Add test coverage for `recalculate_item_total` when line item totals change
- [x] Scope handling of tax adjustments in `InMemoryOrderUpdater` to *not* marked for destruction
- [x] Scope handling of tax adjustments in `OrderUpdater` to *not* marked for destruction
- [ ] Ensure order-level tax adjustments (like Colorado delivery) are scoped out of tax total and adjustment total calculations
- [ ] Handle persistence in `update_promotions`
- [ ] Handle persistence in `update_taxes`
- [ ] Write the `InMemoryOrderAdjuster` (also, should we rename this to `InMemoryOrderPromotionAdjuster`)
Expand Down

0 comments on commit dbc5940

Please sign in to comment.