microsoft / WindowsAppSDK

The Windows App SDK empowers all Windows desktop apps with modern Windows UI, APIs, and platform features, including back-compat support, shipped via NuGet.
https://docs.microsoft.com/windows/apps/windows-app-sdk/
MIT License
3.85k stars 323 forks source link

Split Notification utility helpers into a mutiple purpose-focused files #2307

Open danielayala94 opened 2 years ago

danielayala94 commented 2 years ago

Describe the bug

Currently we have an AppNotificationUtility which contains different helper functions addressing unrelated areas.

We want a common folder which hosts mutiple purpose-focused files containing different sets of helpers.

Steps to reproduce the bug

Code bug

Expected behavior

No response

Screenshots

No response

NuGet package version

No response

Packaging type

No response

Windows version

No response

IDE

No response

Additional context

No response

danielayala94 commented 2 years ago

Also, ConvertUtf8StringToWideString can be moved to the WinAppSDK common Utf8 utility file and maybe add a wil::unique_cotaskmem_string overload.