Francesco Calicchio

Results 29 comments of Francesco Calicchio

@uveshm003 I didn't test the PR yet, but just by looking at the code, I can tell that u didn't make the changes that luca pointed [here](https://github.com/RIP-Comm/sossoldi/pull/300#issuecomment-2727452649). I can see...

> Hey, The calculator image is only displayed in transactions screen not on home screen. I have used a Boolean variable to manage this. You can check the same. You're...

I got some concern about this... If I pressed on the “copy transaction” button, I would end up on the same exact page where I was. Since the pages for...

We will use "traditional" Clean Architecture? With Domain, Data,Presentation, Infrastracture and so on?

Thanks for the PR. Reviewing the `features/` directory, I'm finding the organization unclear. Some subfolders seem to represent pages (`graphs_page`, `settings_page`, `transactions_page`, `dashboard_page`) rather than application features. I would consider...

I'll try to review this, and test on iOS, by tomorrow evening or sunday at max, thanks for the PR!

> I think this could be improved a bit further and make it ready for future settings by making the `SelectorContainer` an expandable tile. I think it's a very cool...

> Let's wait for the new layout to be pushed and you can re-test Wdym by the "new layout" what did I miss?

@theperu Thanks for the review! I don't sincerely understand why a user would ever need to duplicate a recurring transaction, please clarify to me this use case, probably I am...