Add instructor filter to Report + auto-set display name on login#166
Merged
ChristopherRotnes merged 7 commits intoMay 13, 2026
Merged
Conversation
- fetchSessionsForReport now joins trainer_id + profiles(display_name) - Report shows a 4th filter chip row (instructor names) when >1 instructor present in the period; empty selection = all instructors (default) - Reset filter clears instructor selection alongside days/types - ensureDisplayName() sets profiles.display_name to email prefix on first login if not yet set, ensuring the filter always has a meaningful label - Docs updated: CHANGELOG, README, CLAUDE.md https://claude.ai/code/session_01Ks23ragnyERfPjjqHpfyjp
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-island-090dfd003-166.westeurope.7.azurestaticapps.net |
claude-opus-4-5 has been retired by Anthropic; updated CLAUDE_MODEL_VISION in prompts.js and the model allowlist + test in claudeUtils.js to use claude-opus-4-7. https://claude.ai/code/session_01Ks23ragnyERfPjjqHpfyjp
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-island-090dfd003-166.westeurope.7.azurestaticapps.net |
OCR + structured JSON extraction from whiteboard photos doesn't require Opus-level reasoning. Both CLAUDE_MODEL_VISION and CLAUDE_MODEL_TEXT now use claude-sonnet-4-6; allowlist reduced to a single entry. https://claude.ai/code/session_01Ks23ragnyERfPjjqHpfyjp
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-island-090dfd003-166.westeurope.7.azurestaticapps.net |
When Anthropic returns a non-OK response, include the actual error message as a 'detail' field in the JSON body. Server logs now include the error type and message. The UI reads data?.error?.message which is unaffected by this shape, so users still see the generic Serverfeil message — detail is visible only in the network tab for developers. https://claude.ai/code/session_01Ks23ragnyERfPjjqHpfyjp
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-island-090dfd003-166.westeurope.7.azurestaticapps.net |
…lossary - Report instructor filter now derives from gym_calendar.instructor (the sporty.no class leader e.g. "Linda Hatlevik") instead of trainer_id (the app user who logged the session) — these are entirely different concepts; the previous implementation produced a filter that never showed more than one entry for a single-user account - fetchSessionsForReport: removed trainer_id + profiles join, added instructor to gym_calendar select - CSP: added sha256 hash for inline theme script to script-src so the FOUC-prevention script is no longer blocked - i18n: added report.unnamed key to all three locales (was hardcoded "Unnamed" in English only) - CLAUDE.md: added Glossary section defining User, Trainer (avoid), Instructor, Co-instructor, Session, Gym class, Exercise variants, Template, Muscle concepts, and System concepts Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-island-090dfd003-166.westeurope.7.azurestaticapps.net |
ChristopherRotnes
deleted the
claude/clarify-session-reports-visibility-ceio8
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
$(cat <<'EOF'
Summary
fetchSessionsForReportnow joinstrainer_idandprofiles(display_name)so instructor identity is available without an extra query.ensureDisplayName()added todb.jsand called alongsideensureGymMembership()on every login. Ifprofiles.display_nameis null it is set automatically to the prefix before@in the user's email — ensures the instructor filter always has a meaningful label without requiring manual action in Settings. Falls back to"Unnamed"in the filter if somehow still null.Test plan
profiles.display_nameis populated (check Settings → Konto or Supabase dashboard)https://claude.ai/code/session_01Ks23ragnyERfPjjqHpfyjp
EOF
)
Generated by Claude Code