Skip to content

Commit

Permalink
Add TODO so we know what to do
Browse files Browse the repository at this point in the history
  • Loading branch information
forkata committed Nov 8, 2024
1 parent 6d6cc69 commit d244646
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions TODO.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
In-Memory Order Updater TODO
===

- [ ] Finish renaming methods that don't persist ever
- [ ] Consider Sofia's recommendation to break this class into POROs to simplify testing
- [ ] Address FIXME on renaming `recalculate_adjustments`?
- [ ] Add test coverage for `update_item_total` when line item totals change
- [ ] Test coverage to ensure state changes aren't persisted (if someone changes current implementation)
- [ ] Handle persistence in `update_promotions`
- [ ] Handle persistence in `update_taxes`

0 comments on commit d244646

Please sign in to comment.