fix: remove 'opts' argument from cards functions

abrantesarthur commented 3 years ago

O que esse PR faz? (Obrigatório)

remove parametro opts de todas funções do namespace cards

Link / Imagem para referencias (Obrigatório)

Apesar da biblioteca padrão do pagarme.js parecer demonstrar que as funções do namespace card esperam dois arguments (opts e body), o parametro opts parece ser injetado pela própria biblioteca, não sendo necessário passá-lo manualmente. De fato, fazer isso acarreta em comportamento incorreto das funções. O jeito correto é passar somente body, como sugere a documentação de criando-um-cartao.

Não tive a chance de testar se o mesmo é válido para todas as outras interfaces que tem opts e body mas acredito que também seja o caso. Se for, seria interessante remover opts das demais funções também e, inclusive, redefinir a nomenclatura da package para usar body em vez de options sempre que possível. Posso abrir um issue sobre isso.

jonyw4 commented 3 years ago

Atualizei a branch para apontar para a branch next pra podermos juntar os PR que vc abriu e sair tudo em um unico release. Para isso preciso que vc atualiza o seu branch com um rebase

abrantesarthur commented 3 years ago

@jonyw4 pronto!

github-actions[bot] commented 3 years ago

:tada: This PR is included in version 2.9.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: