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.

Sign up for GitHub

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

Jump to bottom

Microsoft.Toolkit.Graph should be .NET Standard? #60

Closed
michael-hawker opened this issue Aug 17, 2020 · 2 comments
Closed

Microsoft.Toolkit.Graph should be .NET Standard? #60

michael-hawker opened this issue Aug 17, 2020 · 2 comments
Labels
bug 🐛 Something isn't working Completed 🔥 enhancement ✨ New feature or request
Milestone
7.1.0

Comments

@michael-hawker
Copy link
Member

It's multi-targeted to UWP currently, but think this can be .NET Standard only?

We need to test if we can just remove this and have MSAL still work?
@michael-hawker michael-hawker added the bug 🐛 Something isn't working label Aug 17, 2020
@ghost ghost added the needs triage 🔍 label Aug 17, 2020
@ghost
Copy link

ghost commented Aug 17, 2020

Hello michael-hawker, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

@ghost ghost added the needs attention 👋 label Sep 1, 2020
@CommunityToolkit CommunityToolkit deleted a comment Nov 17, 2020
@CommunityToolkit CommunityToolkit deleted a comment Nov 17, 2020
@CommunityToolkit CommunityToolkit deleted a comment Nov 17, 2020
@CommunityToolkit CommunityToolkit deleted a comment Nov 17, 2020
@CommunityToolkit CommunityToolkit deleted a comment Nov 17, 2020
@CommunityToolkit CommunityToolkit deleted a comment Nov 17, 2020
@shweaver-MSFT shweaver-MSFT mentioned this issue Mar 23, 2021
Merged
6 tasks
@ghost ghost added the In-PR 🚀 label Mar 23, 2021
@shweaver-MSFT shweaver-MSFT mentioned this issue Mar 24, 2021
Closed
9 tasks
@ghost ghost added Completed 🔥 and removed In-PR 🚀 labels Mar 24, 2021
@shweaver-MSFT
Copy link
Member

shweaver-MSFT commented Mar 25, 2021 •
edited
Loading

This was changed in PR #76.

Now the main CommunityToolkit.Net.Authentication package is purely NetStandard. There is no more multi-targeting in the solution.

@ghost ghost locked as resolved and limited conversation to collaborators Apr 24, 2021
@shweaver-MSFT shweaver-MSFT added this to the 7.1.0 milestone Aug 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug 🐛 Something isn't working Completed 🔥 enhancement ✨ New feature or request
Projects
None yet
Milestone
7.1.0
Development

No branches or pull requests
2 participants
@michael-hawker @shweaver-MSFT