Skip to content

Optional Icon field is added to Goal Model and added support for goal icons and tests#192

Open
Siddhi-Sikchi wants to merge 2 commits into
fencer-so:mainfrom
Siddhi-Sikchi:main
Open

Optional Icon field is added to Goal Model and added support for goal icons and tests#192
Siddhi-Sikchi wants to merge 2 commits into
fencer-so:mainfrom
Siddhi-Sikchi:main

Conversation

@Siddhi-Sikchi

Copy link
Copy Markdown

Summary

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

Backend Changes

  • Added optional icon support to the Goal model.
  • Updated backend services to support goal icon persistence.
  • Added unit tests for the GetGoalsForUser endpoint.

Frontend Changes

  • Updated the Goal model to include icon support.
  • Added emoji picker functionality.
  • Added GoalIcon and AddIconButton components.
  • Updated GoalManager to allow users to add and edit icons.
  • Updated goal cards to display selected icons.

API Changes

  • Added PUT request support to persist icon changes.

Testing

  • Added xUnit test coverage for the GetGoalsForUser route.

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