Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: add wallet connect API support #9

Closed
wants to merge 4 commits into from
Closed

feat: add wallet connect API support #9

wants to merge 4 commits into from

Conversation

coffeeispower
Copy link
Collaborator

@coffeeispower coffeeispower commented Oct 3, 2023 •
edited
Loading

Esse pull request adiciona suporte a api de wallet connect do mercado pago, que permite a conexão da carteira do cliente com a aplicação para pagamentos futuros.

Progresso:

  • Agreements:
    • Criar agreements
    • Criar payer tokens
    • Cancelar agreement
  • Advanced payments:
    • Criar advanced payment
    • Confirmar/Cancelar advanced payment
    • Obter dados de um advanced payment
    • Reembolsar um advanced payment

Closes #7

@coffeeispower coffeeispower linked an issue Oct 3, 2023 that may be closed by this pull request
[Feature] Wallet Connect #7
Closed
@coffeeispower coffeeispower added the enhancement New feature or request label Oct 3, 2023
@coffeeispower
Copy link
Collaborator Author

Só pra avisar que eu ativei o #![warn(clippy::pedantic)] aí, ent eu mudei um pouco o resto do código, mas mudou pra melhor acredito, ent o clippy vai pegar muito no pé só avisando kk

jackskelt
jackskelt requested changes Oct 3, 2023
View reviewed changes
Copy link
Collaborator

@jackskelt jackskelt left a comment

Choose a reason for hiding this comment

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

Falta agora a parte de criar o Payer Token

src/client.rs Show resolved Hide resolved
@coffeeispower coffeeispower force-pushed the 7-feature-wallet-connect branch 2 times, most recently from 57c2708 to 0e57069 Compare October 3, 2023 13:20
@coffeeispower coffeeispower mentioned this pull request Oct 3, 2023
Closed
@coffeeispower
Copy link
Collaborator Author

Acabei de adicionar uma função pra criar o builder dos agreements porque eu tinha esquecido

@coffeeispower coffeeispower removed their assignment Oct 6, 2023
@jackskelt
Copy link
Collaborator

Fechar PR pelo mesmo motivo da issue

#7 (comment)

@jackskelt jackskelt added wontfix This will not be worked on and removed enhancement New feature or request labels Oct 21, 2023
@jackskelt jackskelt closed this Oct 21, 2023
@jackskelt jackskelt deleted the 7-feature-wallet-connect branch October 21, 2023 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jackskelt jackskelt Awaiting requested review from jackskelt

Assignees

@jackskelt jackskelt

Labels
wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature] Wallet Connect
2 participants
@coffeeispower @jackskelt