Skip to content

fix(dashboard): persist Zielnote and make it more compact - #8

Merged
veniplex merged 2 commits into
mainfrom
claude/dashboard-target-grade-bug-jam43e
Jul 13, 2026
Merged

veniplex merged 2 commits into
mainfrom
claude/dashboard-target-grade-bug-jam43e

Conversation

@veniplex

Copy link
Copy Markdown
Owner

Summary

  • The "Zielnote" (target grade) input on the dashboard's semester overview was only kept in component state, so it reset to the default (2.0) every time the user navigated away and back. It's now persisted per program in localStorage.
  • The widget took up a full row as a bordered box. Shrunk it to a compact, single-line, low-emphasis inline control (smaller text/input, no border/box) matching the request that it be more "dezent".

Test plan

  • npx tsc --noEmit
  • npx eslint on changed files
  • Manually verify in the browser: set a target grade, navigate to another page and back, confirm it's retained; visually confirm the control is compact and unobtrusive

Generated by Claude Code

claude added 2 commits July 13, 2026 16:35
Grade goal target was only in component state and reset on every
navigation. Persist it per program in localStorage. Also trim the
widget to a single-line, low-emphasis inline control instead of a
bordered box taking a full row.
Move the grade goal from localStorage to the degree_program table
(new grade_goal column) so it syncs across devices, following the
same ownership/action pattern as the existing grade scale setting.

Copy link
Copy Markdown
Owner Author

Pushed a follow-up commit: the Zielnote is now persisted server-side in degree_program.grade_goal (new column, migration 0028_bouncy_hawkeye.sql), scoped per user via the existing program-ownership checks, instead of localStorage — so it now syncs across devices. Saved via a debounced updateGradeGoal server action mirroring the existing updateGradeScale pattern.


Generated by Claude Code

@veniplex veniplex added the release-candidate Creates a new release and deployment of docker image, once merged. label Jul 13, 2026
@veniplex
veniplex merged commit f3dcf75 into main Jul 13, 2026
2 of 3 checks passed
@veniplex
veniplex deleted the claude/dashboard-target-grade-bug-jam43e branch July 13, 2026 16:41
github-actions Bot added a commit that referenced this pull request Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-candidate Creates a new release and deployment of docker image, once merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants