microsoft/WindowsAppSDK

PushNotificationManager SelfContained scenarios need better error code

Open

#2 416 ouverte le 18 avr. 2022

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

Métriques du dépôt

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

Description

Describe the bug

Currently, the PushNotificationManager APIs return E_FAIL if SelfContained scenario is not supported. A better HRESULT to use would be HRESULT_FROM_WIN32(ERROR_NOT_SUPPORTED).

From this conversation: https://github.com/microsoft/WindowsAppSDK/pull/2389#discussion_r851574133

Steps to reproduce the bug

In unpackaged/SelfContained environment, call into PushNotificationManager APIs and view the returned error code.

Expected behavior

No response

Screenshots

No response

NuGet package version

No response

Packaging type

No response

Windows version

No response

IDE

No response

Additional context

No response

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.