Skip to content

Support icons - #243

Open
AdinathBorse wants to merge 1 commit into
fencer-so:mainfrom
AdinathBorse:icons
Open

Support icons#243
AdinathBorse wants to merge 1 commit into
fencer-so:mainfrom
AdinathBorse:icons

Conversation

@AdinathBorse

Copy link
Copy Markdown

Support Icons

Changes Made

Frontend (commbank-web)

  • types.ts — Added icon: string | null to the Goal interface
  • GoalCard.tsx — Displays the goal's icon on the goal card
  • GoalManager.tsx — Implemented emoji picker; wired pickEmojiOnClick
    to call updateGoalApi so icon changes persist after client refresh

API Layer

  • lib.ts — Added updateGoal() PUT request to PUT /api/Goal/{goalId}

Tests

  • GoalControllerTests.cs — Added GetForUser test to verify the
    GetGoalsForUser route returns goals filtered by the correct user ID

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