MathiSans / capstone_mood-tracker

https://mood-diary-five.vercel.app
3 stars 0 forks source link

V2/barchart #118

Closed NikCrmr closed 1 week ago

vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mood-diary ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 2, 2024 8:48am
NikCrmr commented 2 weeks ago

Thank you Felix for you feedback! We run in the issue that everything is dependent on each other so it was very difficult to refactor, that's why we passed all the huge basket of props down the BarChart. Do you have any examples how to refactor such a logic? We also thought to refactor some logic into extra files, but also there we noticed that it needs data from the Dashboard.js. This dependencies makes it quite hard to refactor the whole thing.