Skip to content

Add Upper/Lower, Full Body, and 5×5 starter plans - #43

Open
iamgroot400 wants to merge 1 commit into
arvids-unavailable:mainfrom
iamgroot400:starter-plans
Open

Add Upper/Lower, Full Body, and 5×5 starter plans#43
iamgroot400 wants to merge 1 commit into
arvids-unavailable:mainfrom
iamgroot400:starter-plans

Conversation

@iamgroot400

Copy link
Copy Markdown

Addresses the "Additional starter plans (upper/lower, full-body, 5×5…)" item in CONTRIBUTING.md's good-first-issues list.

Load starter plan used to hand you one fixed Push/Pull/Legs routine. It now opens a picker with four templates — PPL, Upper/Lower, Full Body, and 5×5 — each built from the same generic spec → routines/week mapping in frontend/src/lib/starter.js, so adding another plan later is a new entry, not new plumbing.

Summary

  • frontend/src/lib/starter.js — 4 starter plans (PPL, Upper/Lower, Full Body, 5×5), real barbell/dumbbell/cable exercises verified against the exercise DB
  • frontend/src/sheets.jsx — new picker sheet (starterPlanSheet) replacing the old single-plan loader
  • frontend/src/views/Home.jsx, Plan.jsx, Settings.jsx — updated to open the picker

Test plan

  • npm test (192 passing)
  • npm run build
  • Clicked through the flow in a dev browser from a fresh guest profile: opened the picker, loaded Upper/Lower, confirmed the week schedule, routines, and exercise names/sets/reps all resolved correctly

Load starter plan used to hand you one fixed Push/Pull/Legs routine.
It now opens a picker with four templates (PPL, Upper/Lower, Full
Body, 5x5), each built from the same generic spec -> routines/week
mapping in lib/starter.js, so adding another plan later is just a
new entry, not new plumbing.

Addresses the "additional starter plans" item in CONTRIBUTING.md's
good-first-issues list.

@mandoshaly5-cell mandoshaly5-cell left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot_20260825-143944~2.jpg

@iamgroot400 iamgroot400 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-_-

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.

2 participants