Skip to content

Add notes field to WorkoutLog - #2462

Open
kmannarbor wants to merge 2 commits into
wger-project:masterfrom
kmannarbor:feature/workout-log-notes
Open

Add notes field to WorkoutLog#2462
kmannarbor wants to merge 2 commits into
wger-project:masterfrom
kmannarbor:feature/workout-log-notes

Conversation

@kmannarbor

Copy link
Copy Markdown

Adds an optional free-text notes field to the WorkoutLog model so users can record per-set observations (form cues, how it felt, etc.). The field is exposed via the existing REST API serializer.

Closes #2437

Proposed Changes

  • wger/manager/models/log.py — add notes = TextField(blank=True, null=True)
  • wger/manager/migrations/0028_add_notes_to_workout_log.py — migration
  • wger/manager/api/serializers.py — expose notes in WorkoutLogSerializer
  • wger/manager/tests/test_log_model.py — two new tests (saved/retrieved, optional)
  • wger/manager/tests/test_log_session_api.py — two new tests (POST with notes, PATCH notes)

A companion PR to wger-project/react adds the UI: a notes input on the log form, a Notes column in the history grid, and a "Last session" hint showing the previous note when logging the same exercise again.

Related Issue(s)

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Code has been formatted to avoid unnecessary diffs (ruff format && isort .)
  • If the feature is big enough or if there are manual steps needed (deployment
    changes etc.), write a small writeup in CHANGELOG.md

kmannarbor and others added 2 commits July 28, 2026 22:47
Adds an optional free-text notes field to the WorkoutLog model so users
can record per-set observations (form cues, how it felt, etc.). The field
is exposed via the existing REST API serializer.

Closes wger-project#2437
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kmannarbor

Copy link
Copy Markdown
Author

Hi @rolandgeider — following up on this and the companion PRs (wger-project/react#1289, wger-project/flutter#1300) for the notes field from #2437.

Looks like the CI/CodeQL workflow runs on all three PRs expired waiting for approval (first-time contributor gate) rather than actually running. Could someone approve the workflow runs so CI can execute? Happy to push a fresh commit to re-trigger if that's easier. Let me know if anything else is needed for review.

@rolandgeider

Copy link
Copy Markdown
Member

hey! I'll take a look at all these prs once we merge 2.7

@kmannarbor

kmannarbor commented Sep 2, 2026 via email

Copy link
Copy Markdown
Author

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.

Feature request: per-set notes field when logging a workout

2 participants