Skip to content

Commit

Permalink
lint notes
Browse files Browse the repository at this point in the history
  • Loading branch information
mrmurilo75 committed Mar 6, 2024
1 parent 72ccaa1 commit 0cdbfa6
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions development_notebook/2024-03-05.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ These models have admin defined and tested for them, but they dont have any func

# Today

Add functionality for accounts to have the current balance.
Add functionality for accounts to have the current balance.

Simply add save on transactions to add their ammounts to or the delta of their amount to the account. These need to account for all possible relevant field changes, as well as delete.

Expand All @@ -18,7 +18,7 @@ Keep balance in Account updated.

## Brute Force solution

Get balance by function, which calculates at call from all transactions.
Get balance by function, which calculates at call from all transactions.

## Improved solution

Expand All @@ -32,7 +32,7 @@ Problem: We still need to deal with (1.) changing transaction's account and (2.)

Solutions:

1. Simply prevent changing. We may give this feature to the user, but delete the transaction and create another on the different account.
1. Simply prevent changing. We may give this feature to the user, but delete the transaction and create another on the different account.
(Can we do this at DB level (prevent a change)? In [[Future Improvements]] we might want to do this all in DB level.)

2. Deletion should not be a problem. Since we are already using soft delete, we can use this to calculate the balance and then actually delete.
Expand Down

0 comments on commit 0cdbfa6

Please sign in to comment.