-
Notifications
You must be signed in to change notification settings - Fork 37
Get this 🎉 Started #3
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
Conversation
michael-hawker
commented
Sep 14, 2019
- Re-integrates PersonView from initial prototype with Provider system aligning better to Microsoft Graph Toolkit
- UWP XAML Provider Wrappers
- Supports Login/Logout
- Quick Sample Test app
|
@azchohfi or @onovotny any ideas what this error is about? I thought we switched to strong naming? Or could this be some weird conflict of mismatching NuGet and MyGet feeds? |
|
We only signed the netstandard and NET Framework targets. We didn’t strong name UWP as it ignores strong names. There may have been another reason that I can’t currently recall. |
|
Thanks for the background @onovotny, I'm wondering why it's complaining then. Very odd. I'll try switching to the MyGet feed for the control package too and see if that makes any difference. I'll need to do that later anyway once Shane's PR goes in on the other repo. |
|
Hey @onovotny, I tried switching the I have the 16299 SDK installed. Have you seen this type of error before? |
|
Suffice it to say, there were a couple of quirks, but the SampleApp was trying to build in the pipeline as well on the native chain and failing (which is something we don't do in the main repo). Had to remove that. Need to try adding a few things back in, like ARM, and see if that works. Will try that in the morning. Not sure if I should jump back in the commit chain and forward-fix from there for clean-up? |
|
Cleaned up and Merged as #4 |