Conversation
| <% end %> | ||
| <% else %> | ||
| <%= render "accounts/accountables/valuations", account: account %> | ||
| <% end %> |
There was a problem hiding this comment.
Will need to come back around and clean a lot of this markup up. Lots of duplication across the accountable views
|
|
||
| assert_equal 1000, new_account.balance | ||
| assert_equal [ 1250, 1000, 1000, 1000 ], new_account.balances.chronological.map(&:balance) | ||
| end |
There was a problem hiding this comment.
Validates bug when there is a transaction at an earlier date than the first valuation
|
I would also argue that this splitting accounts into two modes is even more confusing. Let's say I created the account above, with "today's balance" 1 CZK. I then change my mind and convert this account from balance mode to transactions mode and add some transactions on the same day. The value is not updated because the "today's balance" is still overwriting it. I guess this is intented (although I could again argue that I added transaction after I updated today's balance on the same day) but now user has even lower chance of figuring out what is going on since he thinks his account is in "transactions only" mode. If the transactions and current balance modes still play role in the balance calculation (where current balance gets obviously priority based on date), why hide those two behind "choosing account mode" GUI? |
|
Also if the account is in "balance only" mode, I can still successfully import transactions. However, after import it gets immediately overwritten by new balance entry which was automatically created with the same value as previous entry for some reason. I don't think this behavior enhance user's ability to know what's going on with their balance. |
|
I was just thinking about the same thing today. I believe a solution here might be introducing a third value tracking mode called "Advanced" which would be a combination of Balance and Transaction tracking mode (= basically the old behavior). What do you think @zachgoll? |
This is in the works already! It will just be a general "Activity View" |
|
@zachgoll I just updated my instance and I'm super satisfied with this activity window. It solves all the issues I have written above :) Good job! |


Allows a user to select the "mode" their account's are in:
CleanShot.2024-10-18.at.16.43.59.mp4
Also includes a few bug fixes around syncing.