search

eporsche / daybreak

Simple timesheets and vacation management for small businesses.
MIT License
127 stars 37 forks source link

[Suggestion] - Avoid using facades #22

Closed Nejcc closed 3 years ago

Nejcc commented 3 years ago

I notice u use a lot Facades - if you want to speed up the app I am suggesting use it where you really need :)

eporsche commented 3 years ago

I see this as micro-optimization. There are plenty of places in the code where I can see a better "time investment" / performance benefit ratio like "n+1 issues" or adding an index table for the report calculation.