microsoft / microsoft/OpenAPI.NET.OData
orderby enum should not contain complex types OR should project sub-properties that are not complex types
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- C#
- Estrelas
- 240
- Forks
- 70
- Merge médio
- 7h 59min
- PRs com merge (30d)
- 13
Descrição
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.
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece rastreando como os metadados de GET /me/calendar/events produzem o enum orderby e onde as propriedades complexas ou de entidade são classificadas. Compare os dois comportamentos propostos, incluindo a profundidade de aninhamento e o tratamento de barras para constantes geradas. O trabalho estará concluído quando o projeto tiver uma regra decidida e testada que evite sugerir valores de orderby que o serviço não pode usar.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- csharp
- Domínio
- api
- Tipo de issue
- Funcionalidade
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100