pnp/cli-microsoft365

[Bug] Without the required scope errors returned from 'tenant people profilecardproperty' commands are vague

Offen

#5.629 geöffnet am 03.11.2023

Auf GitHub ansehenContributor Guide
 (14 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (399 Forks)auto 404
bughelp wanted

Repository-Metriken

Stars
 (1.390 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I don't have the required scope added yet to my app reg but the error returned is a bit vague:

image

That's probably because the error object coming in from the Graph has an empty message. :( image not sure how to solve this here... The MS Graph team should make this more descriptive, or we should implement checks based on the HTTP status code.. But that seems a change that would impact a lot of other code.

Originally posted by @martinlingstuyl in https://github.com/pnp/cli-microsoft365/issues/5624#issuecomment-1791486402

Contributor Guide

Research-Richtung
Untersuche, wie die CLI Fehlerantworten der Graph API verarbeitet. Suche nach Mustern, bei denen die Fehlermeldung leer ist, und schlage Fallback Meldungen basierend auf HTTP Statuscodes oder Bereichsanforderungen vor. Überprüfe die vorhandene Fehlerbehandlung in ähnlichen Befehlen.
Tech Stack
typescript
Domain
backendapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60