microsoft/WindowsAppSDK

Add cleanup to AppNotificationManager::Register

Open

#2269 aperta il 15 mar 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)C++ (453 fork)auto 404
Status: In PRarea-Notificationsfeature proposalgood first issue

Metriche repository

Star
 (4628 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Tech stack
Dominio
Tipo issue
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Prerequisiti
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.