darenc9 / FitFuture

https://fit-future.vercel.app
0 stars 0 forks source link

Mara/progress #95

Closed mfunkel closed 1 month ago

mfunkel commented 1 month ago

What has been done?

This PR adds "progress" functionality and includes frontend updates to the Profile Page, as well as backend updates to the History services and routes, and Profile service to enable progress tracking.

Summary Backend

Summary Frontend

Usage

vercel[bot] commented 1 month ago

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

Name Status Preview Comments Updated (UTC)
fit-future ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2024 6:55pm
fit-future-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2024 6:55pm
mfunkel commented 1 month ago

don't think there would be any issues with the changes you made to the workout routes file as you are just modifying the data type before the history object is being created. shouldn't result in any issues, but may just have to verify that things that require access to the history object still work as expected, for example in starting a workout and viewing the recent history. github may complain about a branch conflict, but from looking at your changes i think it should be okay to just use your version as the only change you made is that for loop within the for loop to modify data types

Yeah, while I was implementing the change of data type stored in db, I checked that the start workout's recent history still shows the correct info, so it should be good to go. I resolved the conflicts that github was complaining about, so it should be fine to merge in that regard.