Skip to content

feat: Add optional Icon field to Goal model - #335

Open
Waleed-Abdullah-Abbasi wants to merge 1 commit into
fencer-so:mainfrom
Waleed-Abdullah-Abbasi:main
Open

feat: Add optional Icon field to Goal model#335
Waleed-Abdullah-Abbasi wants to merge 1 commit into
fencer-so:mainfrom
Waleed-Abdullah-Abbasi:main

Conversation

@Waleed-Abdullah-Abbasi

@Waleed-Abdullah-Abbasi Waleed-Abdullah-Abbasi commented Aug 11, 2026

Copy link
Copy Markdown

Summary

This PR adds an optional Icon field to the Goal model, which is the first step toward supporting emojis/icons for goals across the application.

Changes Made

  • ✅ Added public string? Icon { get; set; } to the Goal class in Goal.cs
  • ✅ Updated backend data models to support the new property
  • ✅ No breaking changes – field is optional and nullable

Next Steps

  • Backend PUT endpoint to update icons
  • Frontend UI to pick emojis
  • Unit tests for the GetForUser route

Checklist

  • Code follows project conventions
  • No breaking changes

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