microsoft/WindowsAppSDK

PushNotificationManager SelfContained scenarios need better error code

Open

#2416 aperta il 18 apr 2022

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

Metriche repository

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

Descrizione

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

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.