Skip to content

Feature/goal icons#294

Open
LaxitSompura wants to merge 3 commits into
fencer-so:mainfrom
LaxitSompura:feature/goal-icons
Open

Feature/goal icons#294
LaxitSompura wants to merge 3 commits into
fencer-so:mainfrom
LaxitSompura:feature/goal-icons

Conversation

@LaxitSompura

Copy link
Copy Markdown

Summary

 Adds optional icon support to the Goal model on the backend, plus test coverage for the GetForUser route.

Changes

 - Added `Icon` (optional string) to the Goal model
 - Verified backward compatibility — existing goals without icons still work
 - Added unit test covering GetForUser route (asserts non-null result, correct type, matching UserId)

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