-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update license and icon based on @kyaa-dost's investigation #3777
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
Update license and icon based on @kyaa-dost's investigation #3777
Conversation
Thanks michael-hawker for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌 |
Based on @Kyaa-dost's initial investigation Keeping old and new formats as not all clients support the new formats in all scenarios yet.
db233c4
to
83984d5
Compare
Looks good from NuGet Package Explorer: Saw that VS doesn't support the license display properly yet, so thinking we should keep both properties. Going to try and add that back. Added comment here: NuGet/Home#8000 (comment) |
Keep embedded license file though as well.
Microsoft.Toolkit.Uwp.Notifications/Microsoft.Toolkit.Uwp.Notifications.nuspec
Outdated
Show resolved
Hide resolved
Hit #3775 issue again, merged main to see if we can get a good build again. |
@azchohfi you good with this? |
Ugh.... I don't mind, but its weird to embed the file and not use it. |
Follow-on to replace #3765
Fixes the nuget license/icon file based on #3765, single commit.