search

RIP-Comm / sossoldi

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

sqflite integration #55

Closed mikev-cw closed 1 year ago

mikev-cw commented 1 year ago

Removed sqflite_common_ffi dependency as apparently not used Renamed database class First attempt to implement all tables All models has their CRUD methods creation of demo bank accounts to show on home page

TODOs Rem ffi dependency on tests too (they will not pass)

FedericoBruzzone commented 1 year ago

After there changes, don't create a new commit. You can use --amend command.

FedericoBruzzone commented 1 year ago

would it be good to write some tests?

mikev-cw commented 1 year ago

I need to add at least one other commit for fixing tests

mikev-cw commented 1 year ago

So it seems FFI dependency is needed for tests. I restored pubspec.yaml and resolved all issues.

FedericoBruzzone commented 1 year ago

Can I merge this PR right now?

mikev-cw commented 1 year ago

I'm ok with it. If you too, we can merge