orderby enum should not contain complex types OR should project sub-properties that are not complex types

Ouverte
#613 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
35/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
csharp
Domaine
api

Piste de recherche

Commencez par retracer comment les métadonnées de GET /me/calendar/events produisent l’énumération orderby et où les propriétés complexes ou d’entité sont classées. Comparez les deux comportements proposés, notamment la profondeur d’imbrication et la gestion des barres obliques pour les constantes générées. Le travail est terminé lorsque le projet dispose d’une règle décidée et testée qui évite de suggérer des valeurs orderby que le service ne peut pas utiliser.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

priority:p1 type:bug

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:

  1. not project properties that are complex/entity types
  2. 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:

  1. how many levels of depths should this support?
  2. 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.

Image

Langage dominant
C#
Étoiles
240
Forks
70
Merge moyen
7 h 59 min
PR mergées (30 j)
13

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de microsoft/OpenAPI.NET.OData

Toutes les issues de microsoft/OpenAPI.NET.OData

Issues similaires

Plus d'issues C#

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.