Skip to content

Support icons#220

Open
Pratik0066 wants to merge 1 commit into
fencer-so:mainfrom
Pratik0066:icons
Open

Support icons#220
Pratik0066 wants to merge 1 commit into
fencer-so:mainfrom
Pratik0066:icons

Conversation

@Pratik0066

Copy link
Copy Markdown

Summary

Added Icon property to the Goal model and implemented the GetForUser controller test.

Changes

  • Goal.cs — Added Icon property to support goal icons
  • GoalControllerTests.cs — Added GetForUser test verifying result is not null, each item is assignable from Goal, and each has the expected UserId
  • CommBank.csproj / CommBank.Tests.csproj — Updated target framework to net10.0
  • launchSettings.json — Fixed application URLs to use HTTPS
  • appsettings.json — Added HTTP → HTTPS redirection config

@gitguardian

gitguardian Bot commented Jun 12, 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.

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