-
Notifications
You must be signed in to change notification settings - Fork 372
Implement WinUI3 support for Desktop package #5411
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
base: main
Are you sure you want to change the base?
Conversation
...ent/Microsoft.Identity.Client.Desktop.WinUI3/Microsoft.Identity.Client.Desktop.WinUI3.csproj
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client.Desktop.WinUI3/WebView2WebUi/WebView2WebUi.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client.Desktop.WinUI3/WebView2WebUi/WebView2WebUi.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/ApiConfig/AcquireTokenInteractiveParameterBuilder.cs
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/ApiConfig/AcquireTokenInteractiveParameterBuilder.cs
Outdated
Show resolved
Hide resolved
…microsoft-authentication-library-for-dotnet into dharshanb/winui3Support
…microsoft-authentication-library-for-dotnet into dharshanb/winui3Support
Hi @DharshanBJ - this has been open for a while. Is it still relevant? |
Hi @bgavrilMS , yes, this PR is still relevant, i'm waiting on reviews for this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
...ent/Microsoft.Identity.Client.Desktop.WinUI3/Microsoft.Identity.Client.Desktop.WinUI3.csproj
Show resolved
Hide resolved
@gladjohn - can you pls review this? |
@bgavrilMS , yes i'll be adding the build changes and public documentation as part of my next work items. |
This pull request addresses the business need outlined in AzureAD/microsoft-authentication-library-for-dotnet#4933. The proposed changes are based on the design detailed in the following document: AuthLibrariesApiReview#18287.
I had shared the preview packages in the issue discussion, many of them tested the packages and confirmed that it works for them - #4933 (comment)