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.

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

swapProposals está faltante na ABI e no contrato SwapTwoSteps e SwapTwoStepsReserve #63

Open
evenstaian opened this issue Oct 19, 2023 · 5 comments
Labels
answered This question was answered, waiting for user to close it. question Further information is requested

Comments

@evenstaian
Copy link

A documentação diz haver uma função mapping (swapProposals) de propostas de SwapTwoSteps. Porém essa função não existe na ABI.

Documentação:
https://github.com/bacen/pilotord-kit-onboarding/blob/main/SwapTwoSteps.md#swapproposals

@gabrielsdev
Copy link
Contributor

É bem provável que esse mapping seja privado.

@evenstaian
Copy link
Author

https://github.com/bacen/pilotord-kit-onboarding/blob/main/SwapTwoSteps.md#swapproposals

Acredito fortemente que não. Veja o mapping frozenBalanceOf, por exemplo. Veja também que funções que são herdadas do ERC20 também não são documentadas, então não vejo porque eles documentariam publicamente funções privadas.

@gabrielsdev
Copy link
Contributor

Acredito que nem tudo que esteja documentado seja para o uso geral, mas apenas para entender melhor o comportamento do contrato. Não consta também, por exemplo, o mapping approvedDigitalCurrency deste contrato, imagino que seja também privado. Além disso, não é permitido fazer deploy na rede e, mesmo que fosse, o método construtor dos contratos (pela sua própria natureza) só é chamado uma única vez no deploy, mesmo assim, estão documentados.

@evenstaian
Copy link
Author

Acredito que nem tudo que esteja documentado seja para o uso geral, mas apenas para entender melhor o comportamento do contrato. Não consta também, por exemplo, o mapping approvedDigitalCurrency deste contrato, imagino que seja também privado. Além disso, não é permitido fazer deploy na rede e, mesmo que fosse, o método construtor dos contratos (pela sua própria natureza) só é chamado uma única vez no deploy, mesmo assim, estão documentados.

Entendo seu ponto. Como faria sentido esse mapping ser publico também, vou aguardar uma resposta oficial para fechar o tópico.

@aldenio aldenio added the question Further information is requested label Oct 24, 2023
@aldenio
Copy link
Member

aldenio commented Nov 16, 2023

Prezado @evenstaian,

O mapping está privado atualmente mas passará a ser público em versões futuras.
Obrigado por sua contribuição.

ps: se sua pergunta foi respondida, favor encerrar a issue.

@aldenio aldenio added the answered This question was answered, waiting for user to close it. label Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered This question was answered, waiting for user to close it. question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants