Skip to content

Tokenização de Bandeira#24

Open
lucassgnm wants to merge 2 commits into
edson-nascimento:masterfrom
lucassgnm:master
Open

Tokenização de Bandeira#24
lucassgnm wants to merge 2 commits into
edson-nascimento:masterfrom
lucassgnm:master

Conversation

@lucassgnm

Copy link
Copy Markdown

No description provided.

@edson-nascimento

Copy link
Copy Markdown
Owner

Fala @lucassgnm seria bom criar o arquivo docs/tokenizacao.md é adicionar os exemplos de uso, e depois linkar o arquivo no readme principal.

@lucassgnm

Copy link
Copy Markdown
Author

Fala @lucassgnm seria bom criar o arquivo docs/tokenizacao.md é adicionar os exemplos de uso, e depois linkar o arquivo no readme principal.

Fala @edson-nascimento ! Consegue verificar para mim se ficou bom dessa forma?

Comment thread src/Getnet/API/BrandCryptToken.php Outdated
Comment thread docs/tockenizacao.md Outdated
Comment thread src/Getnet/API/Tokenization.php Outdated
Comment thread src/Getnet/API/Tokenization.php Outdated

@edson-nascimento edson-nascimento left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fazer ajustes

@lucassgnm

Copy link
Copy Markdown
Author

Fala @edson-nascimento , fiz os ajustes solicitados.

  • PR sem conflito com master
  • removida a necessidade de criar $tokenCard no exemplo
  • Card::setNumberToken() aceita network_token_id
  • adicionado teste unitário para conversão do amount em BrandCryptToken

Validei localmente com composer test:unit e composer phpstan.
Consegue revisar novamente e liberar a review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants