Skip to content

Add optional Icon field to Goal model#229

Open
Hr-cd wants to merge 2 commits into
fencer-so:mainfrom
Hr-cd:main
Open

Add optional Icon field to Goal model#229
Hr-cd wants to merge 2 commits into
fencer-so:mainfrom
Hr-cd:main

Conversation

@Hr-cd

@Hr-cd Hr-cd commented Jun 15, 2026

Copy link
Copy Markdown

No description provided.

@Hr-cd

Hr-cd commented Jun 15, 2026

Copy link
Copy Markdown
Author

Add optional Icon field to Goal model

1 similar comment
@Hr-cd

Hr-cd commented Jun 15, 2026

Copy link
Copy Markdown
Author

Add optional Icon field to Goal model

@Hr-cd

Hr-cd commented Jun 16, 2026

Copy link
Copy Markdown
Author

Summary

This pull request adds support for goal icons throughout the application.

Backend Changes

  • Added an optional Icon field to the Goal model.
  • Updated the backend to support storing and returning goal icons.
  • Added unit test coverage for the GetGoalsForUser endpoint.

Frontend Changes

  • Updated the client Goal model to include the icon property.
  • Displayed goal icons on goal cards.
  • Integrated an emoji picker to allow users to add and change icons.
  • Updated GoalManager to persist icon changes.

Networking

  • Added a PUT request for updating goals.
  • Connected the frontend and backend so icon changes persist after refresh.

Testing

  • Verified that goal icons are correctly saved and returned by the API.
  • Added automated test coverage for GetGoalsForUser.

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