theopensystemslab / fairhold-dashboard

https://fairhold-dashboard.vercel.app
0 stars 0 forks source link

types: Can we share `ForecastParameters` instead of just some children? #52

Closed zz-hh-aa closed 3 weeks ago

zz-hh-aa commented 2 months ago

Likely worth doing after https://github.com/theopensystemslab/fairhold-dashboard/pull/44 is reviewed and merged. ✅

A number of classes take a few properties from ForecastParameters into their constructors.

It may be simpler to pass the whole object along to each of these classes. This would likely be more maintainable and flexible.