microsoft/WindowsAppSDK

Add cleanup to AppNotificationManager::Register

Open

#2.269 aberto em 15 de mar. de 2022

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)C++ (453 forks)auto 404
Status: In PRarea-Notificationsfeature proposalgood first issue

Métricas do repositório

Stars
 (4.628 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Pilha de tecnologia
Domain
Tipo Issue
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Prerequisites
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.