bacen / pix-api

API Pix: a API do Arranjo de Pagamentos Instantâneos Brasileiro, Pix, criado pelo Banco Central do Brasil.
https://bacen.github.io/pix-api
2.36k stars 267 forks source link

[Correção] API 2.1.0 rc0 - vencimento e expiração #100

Closed cryptographix closed 4 years ago

cryptographix commented 4 years ago

Algumas dúvidas sobre o YAML v2.1.0-rc0

CobSolicitada:

ninrod commented 4 years ago

o "required:' indica "expiracao" e "dataDeVencimento", mas consta somente "calendario" como subitem.

existe uma relação oneOf entre os dois elementos

ambos são mesmo obrigatórios na solicitação?

Não, são mutuamente excludentes ("oneOf")

o campo "validadeAposVencimento" é novo. Este deve ser enviado na solicitação ou será formado pelo somatório do "expiracao" e "datadeValidade", conforme comentado em outro issue?

É novo. Ele indica os dias corridos após "dataDeVencimento" em que a cobrança ainda poderia ser paga.

cryptographix commented 4 years ago

@ninrod Hi. Na linha 1260 do yaml, você tem: required: ["valor", "chave", "calendario", "expiracao", "dataDeVencimento"] no entanto, expiracao e dataDeVencimento são propriedades "filhas" do calendario e não diretamente abaixo na hierarquia.

Desculpe o meu desconhecimento, mas é permitido referenciar propriedades filhas? Não encontrei definição clara sobre isso no JSON Schemas e OpenAPI.

Não deveria ser "calendario/expiracao" etc?

ninrod commented 4 years ago

@SeanWykes ,

@ninrod Hi. Na linha 1260 do yaml, você tem: required: ["valor", "chave", "calendario", "expiracao", "dataDeVencimento"]

Ok vamos verificar.

ninrod commented 4 years ago

@SeanWykes , esta linha especifica que você mencionou está com campos sobrando, apesar de não afetar o "rendering" da API. Realmente: o expiração e dataDeVencimento serão removidos desta linha "required". Devo publicar uma nova RC em alguns intstantes. Obrigado pelo reporte. Vou aproveitar e verificar se há mais casos.

ninrod commented 4 years ago

corrigido por 49b1f4ff45bc1c1daa49587bc27428574cb27a3c