microsoft / microsoft/OpenAPI.NET.OData
orderby enum should not contain complex types OR should project sub-properties that are not complex types
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C#
- Estrellas
- 240
- Forks
- 70
- Merge medio
- 7 h 59 min
- PR fusionados (30 d)
- 13
Descripción
for GET /me/calendar/events we currently project an enum value of start and start desc.
This ends up failing on the service because the start property is a complex type and cannot be used for ordering.
The conversion library should either:
- not project properties that are complex/entity types
- project sub properties of the complex/entity types which are themselves scalar types (e.g.
start/dateTime desc)
If we go with 2, this brings a couple of questions:
- how many levels of depths should this support?
- what's going to happen for consumers using that metadata to project enums / union types of constants which all of a sudden will see a / character appear, which is reserved in most programing languages?
This also impacts graph explorer as we're suggesting customers things that cannot work.
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
Comienza siguiendo cómo los metadatos de GET /me/calendar/events producen el enum orderby y dónde se clasifican las propiedades complejas o de entidad. Compara los dos comportamientos propuestos, incluida la profundidad de anidamiento y el tratamiento de las barras para las constantes generadas. Se considera terminado cuando el proyecto tiene una regla decidida y probada que evita sugerir valores de orderby que el servicio no puede usar.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp
- Área
- api
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100