Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Atributos latitude e longitude: documentação conflita com a especificação de precisão #85

Closed
danielgoncalves opened this issue Feb 9, 2022 · 3 comments
Labels
bug Something isn't working logistics Something related to the logistics theme merchant Something related to the merchant theme orders Something related to the orders theme

Comments

@danielgoncalves
Copy link

danielgoncalves commented Feb 9, 2022

Os atributos latitude e longitude dos objetos Service.serviceArea.polygon.geoCoordinates estão documentados como tipo number e decimal places <= 5.

Porém, o texto da documentação diz:

"The precision should be at least 5 decimal places"

O que dá a entender que latitude e longitude devem possuir pelo menos 5 casas decimais, o que é o oposto do que está documentado no tipo do atributo.

Correção Sugerida 1

O texto da documentação deve ser corrigido para:

"Precision must be up to 5 decimal places"

Correção Sugerida 2

O tipo dos atributos latitude e longitude deve ser documentado como:

number decimal places >= 5

Referência

@danielgoncalves danielgoncalves changed the title Atributos latitude e longitude documentação conflita com a especificação de precisão Atributos latitude e longitude: documentação conflita com a especificação de precisão Feb 9, 2022
@gui-camargo
Copy link
Collaborator

Olá @danielgoncalves,

Na verdade essa descrição é gerada automaticamente pelo redoc (que renderiza o yaml do openapi).
Na definição do campo eu declaro dessa maneira:

latitude:
                      type: "number"
                      multipleOf: 0.00001
                      description: "Latitude in degrees. Values are restricted to the range [[-90, 90]].\n\n\n\nThe precision should be at least 5 decimal places."

O redoc automaticamente converte esse multipleOf: 0.00001 para a descrição decimal places <= 5.
Estava lendo o comit do redoc que criou esse convert automático e acredito que é um bug do redoc, porque antes o campo era decimals <= X, aí fazia mais sentido.

Vamos levar as suas sugestões para o comite para ver qual o pessoal prefere, mas provavelmente vou ter que tirar esse multipleOf: 0.00001 porque acredito que não consigo alterar a renderização do HTML.

@danielgoncalves
Copy link
Author

mas provavelmente vou ter que tirar esse multipleOf: 0.00001 porque acredito que não consigo alterar a renderização do HTML.

Sim, parece mais correto que seja mesmo "pelo menos 5 casas decimais", daí pra cima.

@gui-camargo
Copy link
Collaborator

Olá @danielgoncalves,
Conforme discutido no comitê, efetuamos a troca do texto para:

The precision should be at least 5 decimal places.
(multipleOf: 0.00001).

Já disponível na documentação.

@gui-camargo gui-camargo added bug Something isn't working logistics Something related to the logistics theme merchant Something related to the merchant theme orders Something related to the orders theme labels Feb 22, 2022
@gui-camargo gui-camargo transferred this issue from Abrasel-Nacional/Open-Delivery-Beta-Test Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working logistics Something related to the logistics theme merchant Something related to the merchant theme orders Something related to the orders theme
Projects
None yet
Development

No branches or pull requests

2 participants