microsoft/WindowsAppSDK

PushNotificationManager SelfContained scenarios need better error code

Open

#2.416 geöffnet am 18. Apr. 2022

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

Repository-Metriken

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

Beschreibung

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

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.