Skip to content

Conversation

@ghost1372
Copy link
Contributor

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@marcelwgn marcelwgn self-assigned this Jul 30, 2025
Copy link
Contributor

@marcelwgn marcelwgn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for making these changes, look good to me. How did you run the app? When I try to run the "WinUIGallery unpackaged" target in VS I am hitting a fatal exception immediately

@ghost1372
Copy link
Contributor Author

Thank you for making these changes, look good to me. How did you run the app? When I try to run the "WinUIGallery unpackaged" target in VS I am hitting a fatal exception immediately

Hi @marcelwgn
You're encountering this exception because the WinUIGallery still uses Application Settings, which aren't supported in unpackaged apps.
The issue is addressed in PR #2001, but it hasn't been merged yet.
Once it's merged, the app should run without issues in unpackaged mode.
I've tested it locally using the changes from that PR, and it worked fine.

Copy link
Contributor

@marcelwgn marcelwgn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the pointer, I tried things out; changes look good to me.

@marcelwgn marcelwgn enabled auto-merge (squash) August 1, 2025 14:09
@marcelwgn
Copy link
Contributor

/azp run

@marcelwgn marcelwgn disabled auto-merge August 1, 2025 14:09
@marcelwgn marcelwgn enabled auto-merge (squash) August 1, 2025 14:10
@marcelwgn marcelwgn merged commit b040ec6 into microsoft:main Aug 1, 2025
2 checks passed
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.

2 participants