bacen / pilotord-kit-onboarding

Documentação e arquivos de configuração para participação no Piloto do Real Digital
905 stars 220 forks source link

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

Open evenstaian opened 1 year ago

evenstaian commented 1 year ago

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 commented 1 year ago

É bem provável que esse mapping seja privado.

evenstaian commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 12 months ago

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.