search

jonyw4 / pagarme-js-types

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

feat(transaction):creating transaction with customer id #42

Closed murilomaiaa closed 3 years ago

murilomaiaa commented 3 years ago

Para criar uma transaction é permitido passar um objeto customer ou apenas o id do customer

jonyw4 commented 3 years ago

Você pode por favor mandar uma referencia da documentação?

jonyw4 commented 3 years ago

E pelo o que entendi essa modificação é no split rules da transação e não diretamente na transação, certo? Se for, a descrição do pull request esta incorreta.

jonyw4 commented 3 years ago

Esse repositório usa commitlint e husky para manter um padrão nos commits. Percebi que seus commits estão com / logo depois do scope. Por favor, peço que leia como funciona o padrão de commit que estamos usando no projeto para que você possa usar corretamente. No caso se você trocar a / para :, remover os traços já iria resolver o problema.

Iria ficar assim: 

feat(transaction): create transaction with customer id
jonyw4 commented 3 years ago

@mrlmaia Eu marquei o PR como draft pra você poder corrigir e me marquei para revisão para quando você finalizar de corrigir e enviar os itens pendentes.

murilomaiaa commented 3 years ago

@jonyw4 Achei esse exemplo nessa PR, testei e funcionou. Agora, para alterar os commits eu nao sei como faço, meu conhecimento em git não é dos melhores. Se você puder mudar isso, vai ser de grande ajuda

jonyw4 commented 3 years ago

Eu vou dar merge nesse PR. Nos proximos você envia de acordo com que tinha comentado

github-actions[bot] commented 3 years ago

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

The release is available on:

Your semantic-release bot :package::rocket: