search

RIP-Comm / sossoldi

"Sossoldi" is a wealth management / personal finance / Net Worth tracking app, made with Flutter.
MIT License
262 stars 73 forks source link

Transactions working implementation + minor fix #80

Closed lucaantonelli closed 1 year ago

lucaantonelli commented 1 year ago

Now transactions works fine, insert/update/delete and income/expense/transfer. The only missing implementation is for recurring payment.

Known Issue: A glitch when disposing the transaction modal on delete

theperu commented 1 year ago

While checking this I noticed a couple of things:

  1. It looks like we don't have any categories from the add transaction page which to me it doesn't seem right
  2. When I try to add a transfer between two accounts and I come back to the dashboard I see an error Not sure if this is specifically related to this PR but we should look into that. Also if I close and reopen the app the error for the account transfer goes away kinda like it wasn't saved. Am I doing something wrong? I uploaded a video of my tests here
lucaantonelli commented 1 year ago

While checking this I noticed a couple of things:

  1. It looks like we don't have any categories from the add transaction page which to me it doesn't seem right
  2. When I try to add a transfer between two accounts and I come back to the dashboard I see an error Not sure if this is specifically related to this PR but we should look into that. Also if I close and reopen the app the error for the account transfer goes away kinda like it wasn't saved. Am I doing something wrong? I uploaded a video of my tests here

Have you tried removing app data or delete the app and reinstall? It seems you could have an old version of the "fake data" in the db, otherwise if it continues to not work can you share the console log?

theperu commented 1 year ago

You are right, it looks like something went wrong with the iOS emulator and I didn't notice. I tried again and it looks fine now. Great work! To me it looks like we can merge this if you are not planning to implement the recurring payment right away

mikev-cw commented 1 year ago

I confirm it's working on Android too. Super cool, let's merge! This contributes to #50