Skip to content

Update toast-notifications.md#5402

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

Merged
merged 2 commits into from
Apr 21, 2025
Merged

Conversation

Max-Meng
Copy link
Contributor

we should check whether args.Argument.Contains("action") before performing the switch check, otherwise, when user click on an empty area of the notification, no such argument is passed, and the application will crash.

we should check whether args.Argument.Contains("action"), otherwise, when user click on an empty area of the notification, no such argument is passed, and the application will crash.
@prmerger-automatorPRMerger Automator
Copy link
Contributor

@Max-Meng : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prodLearn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit 0d62b24:

✅ Validation status: passed

FileStatusPreview URLDetails
hub/apps/windows-app-sdk/migrate-to-windows-app-sdk/guides/toast-notifications.md✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prodLearn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit 85e91e6:

✅ Validation status: passed

FileStatusPreview URLDetails
hub/apps/windows-app-sdk/migrate-to-windows-app-sdk/guides/toast-notifications.md✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@stevewhimsstevewhims merged commit 4c96a79 into MicrosoftDocs:docsApr 21, 2025
2 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment
close