chore(Commerce): Update version and UnoFeatures - #891
Conversation
chore: Update Windows PublishProfiles to the Dir where the current templates store them Packages has been referencing Refit, but we only had http as feature Should be checked, if MacCatalyst is still a valid Platform to have in the Platforms folder and TargetFrameworks list
There was a problem hiding this comment.
Pull Request Overview
This PR updates the Commerce reference application to use a newer version of the Uno SDK and refines its feature configuration. The changes focus on modernizing the SDK version, improving UnoFeatures configuration, and updating build settings.
- Updated Uno.Sdk from version 6.0.67 to 6.1.23
- Added HttpRefit feature and commented out ExtensionsCore feature
- Updated target framework ordering and resource copy settings
Reviewed Changes
Copilot reviewed 2 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| reference/Commerce/src/global.json | Updates Uno.Sdk version and removes outdated comment |
| reference/Commerce/src/Commerce/Commerce.csproj | Adds HttpRefit feature, comments out ExtensionsCore, reorders target frameworks, and changes resource copy behavior |
Sorry, something went wrong.
| Navigation; | ||
| <!--ExtensionsCore;--> | ||
| SkiaRenderer; |
Copilot
AI
Jul 31, 2025
There was a problem hiding this comment.
[nitpick] Consider removing the commented-out ExtensionsCore feature entirely rather than leaving it as a comment, as this reduces code clutter and confusion about whether it should be enabled.
Sorry, something went wrong.
There was a problem hiding this comment.
Team should verify its not longer needed please
Sorry, something went wrong.
There was a problem hiding this comment.
@Jen-Uno do we want to remove this entirely even if its available?
e.g. I am frequently using it in Library Apps managed by the Uno.Sdk for the Uno Specialized Collection/Enumerable Extensions to have the Uno.Sdk automatically add the nuget to my project 🤔 Would be really sad if Uno really wants to remove this Feature entirely 🙁
Sorry, something went wrong.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.