microsoftgraph / microsoftgraph/msgraph-sdk-php
missing models for ChangeNotification & ChangeNotificationCollection
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 669
- Forks
- 150
- Ø Merge
- 15 Std. 21 Min.
- Gemergte PRs (30 T.)
- 3
Beschreibung
Describe the bug
I'm unable to find the ChangeNotification & ChangeNotificationCollection models in version 2 of the msgraph-sdk-php.
These 2 models are included in version 1.
https://github.com/microsoftgraph/msgraph-sdk-php/tree/1.110.0/src/Model
These classes appear to have been left out of other versions of the graph sdk, but were fixed:
https://github.com/microsoftgraph/msgraph-sdk-go-core/issues/95
I also looked at a C# implementation for coding hints, looked at similar models in the source code for various models, and looked through Kiota documentation for deserialization, but wasn't able to find a straightforward way to handle change notifications using the graph api sdk. If more documentation & examples could be added to the SDK, that would be helpful.
Expected behavior
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Vergleichen Sie das im Issue verlinkte src/Model-Verzeichnis von Version 1 mit den generierten Modellen im aktuellen SDK, untersuchen Sie anschließend ähnliche Modelle und die Kiota-Dokumentation zur Deserialisierung. Bestätigen Sie, wie ChangeNotification und ChangeNotificationCollection dargestellt und verfügbar gemacht werden sollten, und überprüfen Sie, ob die erwarteten PHP-Imports für die Verarbeitung von Änderungsbenachrichtigungen verfügbar und verwendbar sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100