Skip to content

Add goal icon support across application#218

Open
raghavthaman wants to merge 1 commit into
fencer-so:mainfrom
raghavthaman:add-goal-icon-support
Open

Add goal icon support across application#218
raghavthaman wants to merge 1 commit into
fencer-so:mainfrom
raghavthaman:add-goal-icon-support

Conversation

@raghavthaman

Copy link
Copy Markdown

Implemented support for goal icons.

Changes:

  • Added optional Icon property to Goal backend model.
  • Updated frontend Goal model to support icons.
  • Added emoji picker support.
  • Displayed icons on goal cards.
  • Added PUT request to persist icon changes.
  • Added xUnit tests covering GoalController icon support.

Testing:

  • Verified goals load successfully.
  • Verified icon changes persist after refresh.
  • Verified backend tests pass.

@gitguardian

gitguardian Bot commented Jun 11, 2026

Copy link
Copy Markdown

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@raghavthaman raghavthaman force-pushed the add-goal-icon-support branch from aca9c7e to bc37379 Compare June 11, 2026 17:20
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