microsoftgraph / microsoftgraph/msgraph-sdk-php

Feedback of Version 1 and Version 2

Offen
#1,427 8 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Needs: Attention :wave: v2: feedback
Vorherrschende Sprache
PHP
Sterne
669
Forks
150
Ø Merge
15 Std. 21 Min.
Gemergte PRs (30 T.)
3

Beschreibung

With reference to the version description https://github.com/microsoftgraph/msgraph-sdk-php/blob/dev/UPGRADING.md and request for comments regarding the new syntax.

I don't know what the consensus was but my 2 cents would be to support both (although looking at the source code it looks like the v1 syntax is no longer supported) . My opinion we should remain as close to the graph endpoints as possible, abstracting away from the ms-graph endpoints makes the interfacing very subjective and far from obvious - I hope this is not too late. The ability to modify attributes to adapt to different scenarios has to be easier with the V1 as well - I would have thought.

// v1.x
$response = $graphClient->createRequest('GET', '/users/userId/messages')
->setReturnType(Model\User::class)
->execute();
// v2.0
$response = $graphServiceClient->users()->byUserId('userId')->messages()->get()->wait();

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit UPGRADING.md und vergleiche die in der Issue zitierten Request-Beispiele für v1 und v2. Dieses Feedback benötigt eine Maintainer-Entscheidung zur Syntaxkompatibilität und zur Anpassung von Attributen, bevor der Umfang der Implementierung festgelegt werden kann; abgeschlossen wäre es, wenn eine abgestimmte Richtung und die entsprechende Aktualisierung oder Änderung vorliegen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
api
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
20/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.