microsoftgraph / microsoftgraph/msgraph-sdk-php
missing models for ChangeNotification & ChangeNotificationCollection
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 669
- Forks
- 150
- Merge medio
- 15 h 21 min
- PR fusionados (30 d)
- 3
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Compare el directorio src/Model de la versión 1 enlazado en el issue con los modelos generados en el SDK actual y, a continuación, inspeccione modelos similares y la documentación de deserialización de Kiota. Confirme cómo deben representarse y exponerse ChangeNotification y ChangeNotificationCollection, y verifique que los imports de PHP esperados estén disponibles y puedan utilizarse para la gestión de notificaciones de cambio.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- api
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100