microsoft/WindowsAppSDK

Add cleanup to AppNotificationManager::Register

Open

#2.269 geöffnet am 15. März 2022

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (453 Forks)auto 404
Status: In PRarea-Notificationsfeature proposalgood first issue

Repository-Metriken

Stars
 (4.628 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Tech Stack
Domain
Issue Type
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Voraussetzungen
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.