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

Flexible routines #1827

Draft
wants to merge 115 commits into
base: master
Choose a base branch
from
Draft

Flexible routines #1827

wants to merge 115 commits into from

Conversation

rolandgeider
Copy link
Member

@rolandgeider rolandgeider commented Nov 21, 2024

Backend changes for the flexible routines

wger-project/react#975
wger-project/flutter#682

This is just a first version, will probably need to work a bit more on it
While this is not 100% correct, it is a clearer name
This can be used for example to label the different date ranges or weeks, or set
other info such as "deload", etc.
This has only the information the client needs to render the workout day
This field is now a charfield so that new options can be added
This allows us to know which entries were part of a dropset or similar
# Conflicts:
#	wger/core/templates/tags/render_day.html
#	wger/manager/tests/test_day.py
#	wger/manager/urls.py
#	wger/manager/views/day.py
#	wger/manager/views/set.py
# Conflicts:
#	wger/core/templatetags/wger_extras.py
#	wger/manager/views/day.py
This is clearer and is actually what the value currently represents.
# Conflicts:
#	wger/core/static/react/main.js
#	wger/manager/helpers.py
#	wger/manager/urls.py
#	wger/manager/views/log.py
This allows a setting to be automatically repeated until a new one appears and
allows the user to confige e.g. an increasing weekly weight with only one entry.
This doesn't make much sense for such a field anyway
In this case, the computed value will be returned as-is
This makes the behaviour more consistent with the rest of the configs, even
if practically this won't be used as much
This is basically a NULL option for the first iteration
This will allow to store the requirements needed for the rule to apply
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant