Skip to content

Conversation

@agniuks
Copy link
Contributor

@agniuks agniuks commented Nov 18, 2021

Description

Microsoft.ReactNative.WindowsAppSDK building on WinAppSDK 1.0.0 🎉 This Includes a template app (C#, MSIX packaged) and the E2E CLI logic for said app to be generated and built using the --usewinui3 flag, currently needs to be deployed from VS.

Closes:

Partially addresses:

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
Microsoft Reviewers: Open in CodeFlow

@agniuks agniuks requested a review from asklar February 11, 2022 23:28
@agniuks agniuks linked an issue Feb 24, 2022 that may be closed by this pull request
2 tasks
@ghost ghost added the Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) label Mar 16, 2022
@ghost ghost removed the Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) label Mar 17, 2022
@agniuks agniuks requested a review from asklar March 18, 2022 18:52
)
return importProjectExists(
projectContents,
'Microsoft.ReactNative.Uwp.CSharpApp.targets',
Copy link
Member

Choose a reason for hiding this comment

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

looks like you removed the check for importing Microsoft.ReactNative.WinAppSDK.CSharpApp.props, was that intended?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I ended up reverting to the logic we had previously because the check for importing WASDK started failing and so the CLI wouldn't build. Using this for now as it works, and plan to update/try checking for a targets file like UWP does once we add a targets file for WASDK.

@agniuks agniuks merged commit 6ba68b3 into microsoft:main Mar 21, 2022
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.

RNW supports WinAppSDK – Phase 0 “Prototype”

4 participants