Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add debug manifest file for development #989

Merged
merged 5 commits into from
Jul 3, 2022

Conversation

marcelwgn
Copy link
Collaborator

Similar to the WinUI 2 gallery, this PR adds a debug manifest that allows you to have the Store version and the development versions installed at the same time instead of the two versions overriding each other all the time.

Question: Right now the app would still display WinUI 3 Gallery when being started, is that good?

xmlns:uap3="http://schemas.microsoft.com/appx/manifest/uap/windows10/3"
xmlns:rescap="http://schemas.microsoft.com/appx/manifest/foundation/windows10/restrictedcapabilities"
IgnorableNamespaces="uap mp uap3">
<Identity Name="Microsoft.WinUI3ControlsGallery.Debug" Publisher="CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US" Version="1.3.13.0" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: use Dev instead of Debug in both the name and filename. Following the Terminal model, we can introduce branding (not necessarily in this PR) so that "WinUI 3 Gallery" is always Release, "WinUI 3 Gallery Dev" is the default and can be Debug or Release

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I see what you are going after here. So would it be better to rename this to dev and make it the default for both debug and release and keep the "non-dev" version up just for building the store version?

@marcelwgn marcelwgn requested a review from bpulliam June 17, 2022 07:57
WinUIGallery/Package.WAP.Dev.appxmanifest Outdated Show resolved Hide resolved
WinUIGallery/Package.WAP.Dev.appxmanifest Outdated Show resolved Hide resolved
@marcelwgn marcelwgn merged commit dde3418 into microsoft:main Jul 3, 2022
@marcelwgn marcelwgn deleted the dev/debug-wap-manifest branch July 3, 2022 09:27
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.

None yet

4 participants