microsoft/WindowsAppSDK

Add cleanup to AppNotificationManager::Register

Open

#2 269 ouverte le 15 mars 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)C++ (453 forks)auto 404
Status: In PRarea-Notificationsfeature proposalgood first issue

Métriques du dépôt

Stars
 (4 628 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Need to add scope_exit after these registrations to remove them in the case of subsequent failure.

Originally posted by @hulumane in https://github.com/microsoft/WindowsAppSDK/pull/2267#discussion_r827388952

Guide contributeur

Direction de recherche
Stack technique
Domaine
Type d'issue
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Prérequis
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.