search

RIP-Comm / sossoldi

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

Error in SQL query in `TransactionMethods.selectById()` #151

Closed jackrua closed 4 months ago

jackrua commented 5 months ago

While working on #149, I have noticed that this query returns an error:

https://github.com/RIP-Comm/sossoldi/blob/366238ecbd4800381d41f73ac5b336da690fc42a/lib/model/transaction.dart#L195-L204

The fix is easy but I would rather close this issue before moving on to push the whole PR.

theperu commented 5 months ago

IMHO you can open just one PR with both fixes, maybe you could have separated commits and make it also clear in the description of the PR itself