Skip to content

Implement Double Progression and Rule Gating Support - #1360

Open
pankaj-basnet wants to merge 2 commits into
wger-project:masterfrom
pankaj-basnet:feat-double-progresssion-reps
Open

pankaj-basnet wants to merge 2 commits into
wger-project:masterfrom
pankaj-basnet:feat-double-progresssion-reps

Conversation

@pankaj-basnet

Copy link
Copy Markdown
Contributor

This pull request Implement Double Progression routines in the Flutter application, matching recent backend capabilities


File Changes Summary

# File Path Change Type Summary
1 lib/features/routines/models/requirement_rules.dart Added Added model definitions to parse and handle double progression requirement rules and flags (all_sets).
2 lib/features/routines/widgets/slot.dart Modified Updated ProgressionRulesInfoBox to dynamically render specific double progression guidelines instead of generic text.
3 test/features/routines/models/requirement_rules_test.dart Added Added unit tests covering requirement rules serialization and parsing logic.
4 test/features/routines/models/slot_entry_model_test.dart Modified Added regression tests
5 test/features/routines/widgets/forms/slot_test.dart Modified Updated widget tests for the routine slot form elements under double progression rules.

Related Issue(s)

wger-project/wger#2480

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Set a 100-character limit to avoid white space diffs (run dart format .)

- add ConfigRequirements and RequirementRule models to parse backend progression constraints (max_repetitions, all_sets). - Update ProgressionRulesInfoBox to dynamically display specific double progression guidance
@pankaj-basnet
pankaj-basnet force-pushed the feat-double-progresssion-reps branch from 07e8422 to 7306740 Compare September 8, 2026 06:17
Remove redundant duplicate evaluation of maxWeightConfigs.length > 1 from the hasProgressionRules getter. Add a regression test to verify evaluation correctness.
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