Skip to content

Support icons#111

Open
fortune-c wants to merge 2 commits intofencer-so:mainfrom
fortune-c:icons
Open

Support icons#111
fortune-c wants to merge 2 commits intofencer-so:mainfrom
fortune-c:icons

Conversation

@fortune-c
Copy link

  1. Modified Goal model to include an optional Icon string field.
  2. Updated the database seeding to include icons for existing goals.
  3. Added a PUT endpoint to update the Icon for a goal.
  4. Verified that API responses include icons when present.
  5. Verified using Postman that emoji changes persist after refresh.
  6. Added unit tests for the GetGoalsForUser endpoint to ensure coverage.

@gitguardian
Copy link

gitguardian bot commented Feb 22, 2026

️✅ 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.

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