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

Improvements to "Transactions" page #84

Closed GBergatto closed 1 year ago

GBergatto commented 1 year ago

I improved the List and Categories tabs. Here are two screenshots of how they look now.

Currently, all transactions are queried from the database regardless of their date. We should query only the ones in the selected time frame. Also, the totals for the day and for each category are calculated in the build method of the widgets, which is not ideal. It might be better to retrieve them directly from the database with a designated query. The same goes for the groupings of transactions in their categories in the "Categories" tab.

The list of expandable categories in the "Categories" tab is wrapped in a Sizebox whose size is calculated every time a category is expanded. While a category gets collapsed, a RenderFlex overflowed is sometimes thrown, because the new size is computed before the end of the animation. After the animation is over, everything is fine. If you can think of a better solution, please tell me.

What's missing

GBergatto commented 1 year ago

Thanks for this PR, in addition to the error already shown on discord, I noticed that some comments are written with lowercase letter. In the moment when you'll update the PR, please change this too.

The error shown on discord is caused by a typing error at line 149 of the file lib/database/sossoldi_database.dart

(14, "Leisure", "subscriptions", '', null, '${DateTime.now()}', '${DateTime.now()}');
15, "Salary", "work", '', null, '${DateTime.now()}', '${DateTime.now()}');

The semicolon at the end of the line should be a comma.

I have fixed this on my local fork but haven't committed the change as I thought it would have been out of scope for this PR. @lucaantonelli said he was fixing a few issues with the database.

If you want me to, I can make all comments start with uppercase letters. However, I think it would be a bit overkill, considering that, as far as I know, no convention about comments has been established. (Maybe we should establish one now)

mikev-cw commented 1 year ago

Thanks for this PR, in addition to the error already shown on discord, I noticed that some comments are written with lowercase letter. In the moment when you'll update the PR, please change this too.

The error shown on discord is caused by a typing error at line 149 of the file lib/database/sossoldi_database.dart

(14, "Leisure", "subscriptions", '', null, '${DateTime.now()}', '${DateTime.now()}');
15, "Salary", "work", '', null, '${DateTime.now()}', '${DateTime.now()}');

The semicolon at the end of the line should be a comma.

I have fixed this on my local fork but haven't committed the change as I thought it would have been out of scope for this PR. @lucaantonelli said he was fixing a few issues with the database.

If you want me to, I can make all comments start with uppercase letters. However, I think it would be a bit overkill, considering that, as far as I know, no convention about comments has been established. (Maybe we should establish one now)

Agreed. Just fixed the typo with a direct commit.