Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calendar isn't showing the couloured (weight) logs #131

Closed
voszp opened this issue Feb 6, 2015 · 5 comments
Closed

Calendar isn't showing the couloured (weight) logs #131

voszp opened this issue Feb 6, 2015 · 5 comments
Assignees
Milestone

Comments

@voszp
Copy link

voszp commented Feb 6, 2015

After creating a (weight) log the calendar is expected to highlight that certain date in green, orange or red.

@rolandgeider
Copy link
Member

Hi @voszp, thanks for the report. How are you adding the logs? I just did some tests and it correctly marks the day in the calendar. By the way, are you using a local installation or the one at wger.de?

@rolandgeider
Copy link
Member

Could reproduce this. The technical reason is that if there is a WorkoutSession entry without any WorkoutLog, nothing is shown in the calendar.

@rolandgeider rolandgeider self-assigned this Feb 7, 2015
@rolandgeider rolandgeider added this to the 1.6 milestone Feb 7, 2015
@voszp
Copy link
Author

voszp commented Feb 7, 2015

I'm using both the online version and a selfhosted dev-version running on an arm machine with Ubuntu Server.
I also found another reason for missing WorkoutLog entries:
Creating a WorkoutLog and leaving the weights blank won't work as well. It's a bit weird entering weights after doing a bodyweight workout.
This can be prevented by setting at least one weight field to zero. Maybe it's possible to set blank fields from bodyweight excercises automatically to zero?

@rolandgeider
Copy link
Member

Nice to see that the app works with arm as well.

Yes, logging "zero weight" is a bit weird. On the other hand for some exercises it's possible to add additional weights (chains, etc.) so having a number here is OK. I personally use the timer view a lot and there the last entry (including "0") is used as a pre-selection.

@rolandgeider
Copy link
Member

@voszp The bug with the calendar highlighting is fixed in master, in case you want to update your local installation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants