fix: update cnpj doc #509
fix: update cnpj doc #509
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@RodriAndreotti pronto pra review |
|
@RodriAndreotti deu http 500 num E2E dentro do tests/fipe-v1.test.js . Consegue dar um re-run? |
Infelizmente não consigo, tenta fazer um amend no PR, teoricamente ele vai rodar de novo. |
|
Aparentemente o retorno de um dos endpoints mudou, acho que vamos precisar corrigir o teste correspondente. |
|
@vhs94 Conseguiria ajustar esse teste para o PR passar? Acredito que podemos substituir esse Desta forma, se o retorno mudar novamente, a gente mantém o teste funcionando. |
| "opcao_pelo_mei": { | ||
| "type": "boolean" | ||
| }, | ||
| "data_opcao_pelo_mei": { | ||
| "type": "string", | ||
| "format": "date" |
There was a problem hiding this comment.
@LorhanSohaky fiquei com duvida nesse cara ai, posso colocar type date e format nullable?
"type": "date",
"format": "nullable"
There was a problem hiding this comment.
@vhs94 Acho que neste caso daria para usar um anyOf, semelhante ao sugerido nesta resposta aqui:
https://stackoverflow.com/a/70658334
O que acham?
There was a problem hiding this comment.
Confesso que não conheço muito, mas talvez seja possível colocar quais são os campos obrigatórios. Mas a sugestão do Rodrigo parece melhor
| }, | ||
| "data_exclusao_do_mei": { | ||
| "type": "string", | ||
| "format": "date" |
|
Kudos, SonarCloud Quality Gate passed!
|
feito man @RodriAndreotti |
update docs de acordo com o response do endpoint abaixo: