Win32 scheduled notification and Setting fixes

[andrewleader]

Fixes #3626

Identity-less Win32 apps had the following problems...

• If AUMID was longer than 129 characters, exception was thrown when using scheduled toast APIs
• If app has never shown a toast notification yet, scheduled toast notifications won't appear
• If app has never shown a toast notification yet, accessing the Setting property threw an exception
• Removing toast by just tag throw an exception (overload with AUMID requires group to be specified too)

To fix each of these (all fixes specific only to identity-less Win32 apps)...

• Hashed auto-generated AUMID if it was too long
• When AddToSchedule or Setting is called, first checks if has sent a toast before, and if not sends a silent toast and then immediately removes it, so that scheduled toasts and accessing setting value "just work" to the developer
• When sending toasts, if tag is specified but group isn't, assigns a default "empty group" value so that later when removing by only tag, can remove by only tag

This required introducing ToastNotifierCompat (in order to intercept the Show/AddToSchedule API calls). It's merely a copy of ToastNotifier just so that we can add in additional code before/after some specific methods are called.

Additionally, added a code sample to the Win32 sample app for using progress bar within toasts!

PR Type

What kind of change does this PR introduce?

• Bugfix
• Sample app changes

What is the current behavior?

See above

What is the new behavior?

See above

PR Checklist

Please check if your PR fulfills the following requirements:

• Tested code with current supported SDKs
• Pull Request has been submitted to the documentation repository instructions. Link:
• Sample in sample app has been added / updated (for bug fixes / features)
  • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
• Tests for the changes have been added (for bug fixes / features) (if applicable)
• Header has been added to all new source files (run build/UpdateHeaders.bat)
• Contains NO breaking changes

Note that this is NOT a breaking change... while we've changed ToastNotificationManagerCompat.CreateToastNotifier()'s return type, ToastNotificationManagerCompat is only in the 7.0 preview right now (and the first version with that class was only introduced 4 days ago).

Other information

[ghost]

Thanks andrewleader 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 🙌

[andrewleader]

Hey @michael-hawker, this bug fix (plus some more sample app code) is ready for review whenever you get a chance!

[ghost]

Hello @michael-hawker!

Because this pull request has the auto merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

[michael-hawker]

@msftbot merge if @azchohfi approves.

[ghost]

Hello @michael-hawker!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

• I'll only merge this pull request if it's approved by @azchohfi

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".

[andrewleader]

@azchohfi this is ready to be reviewed!

[Kyaa-dost]

@andrewleader Nice work on the PR. Changes in the sample app looking great 🚀🚀

[Kyaa-dost]

@msftbot only merge this PR when approved by @azchohfi

[ghost]

Hello @Kyaa-dost!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

• I'll only merge this pull request if it's approved by @azchohfi

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".