rtghan / FinancialOrganizer

3 stars 0 forks source link

Add income #2

Closed ErnestYuen closed 10 months ago

rtghan commented 10 months ago

Everything looks pretty good, and in terms of naming and structure, remains pretty consistent with our other code. Although it lacks comments/documentation, it's fairly clear. I think the packaging structure we use could probably be updated, because the view and perhaps the data access objects are things that are shared between our use cases, but that is something for the future (along with tests).