jonyw4 / pagarme-js-types

📖 A simple typescript definition file for pagarme-js repository
Apache License 2.0
45 stars 14 forks source link

Porque Document estende Address? #29

Closed italodeandra closed 4 years ago

italodeandra commented 4 years ago

Olá pessoal, tudo bem?

Muito obrigado por compartilharem os types do pagarme. Atualmente estou implementando em meu projeto e estou tendo alguns problemas, irei postar aqui aos poucos e caso concordem que seja um problema eu poderei contribuir caso estejam ocupados.

Mas enfim, pq Document estende Address?

https://github.com/jonyw4/pagarme-js-types/blob/9f00a58fb89317d92b8c56fe921c46ba3fd28172/index.d.ts#L489

Sendo que a API aceita apenas os campos type e number. Estou tendo que contornar isso convertendo o tipo.

Abraços.

jonyw4 commented 4 years ago

Fala @italodeandra. Eu acabei criando esse repo a partir de uma discussão que teve la no repositório do pagar.me, infelizmente eles não estão mantendo a tipagem do projeto (o que é uma pena). Toda e qualquer contribuição é bem-vinda! Criei o repositório para facilitar a manutenção desse código com todo mundo mesmo! 😁

Sobre a sua pergunta, a resposta é: eu não sei. 😂 Eu acredito que ou a documentação estava errada, ou eu me confundi na hora de criar essa interface (o que é mais provável 😂). Mas realmente um Document não estende um Address, então é um bug. Se você puder mandar um PR eu agradeço e lerei com muito prazer!

github-actions[bot] commented 4 years ago

:tada: This issue has been resolved in version 2.3.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: