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

Feature/msig #649

Closed
wants to merge 27 commits into from
Closed

Feature/msig #649

wants to merge 27 commits into from

Conversation

henriquecbuss
Copy link
Member

What issue does this PR close

Closes #639

Changes Proposed ( a list of new changes introduced by this PR)

  • Adicionar abilidade de configurar multisig, e usá-la pra propor, votar e executar transações EOS

How to test ( a list of instructions on how to test this PR)

Por enquanto, tudo está hardcoded no módulo Page.Community.Settings.Multisig. Se quiser testar, mude os valores nesse arquivo para usar suas contas.

  • Acesse /community/settings/multisig
  • Configure as contas que receberão permissão para propor, votar e executar transações
  • Proponha uma transação
  • Vote na transação até ela estar aprovada (2 votos)
  • Execute a transação

src/elm/Api/Eos.elm Outdated Show resolved Hide resolved
@lucca65
Copy link
Member

lucca65 commented Mar 25, 2022

Bro, I think we should update this PR with our master branch, maybe there is still time so the merge won't be too crazy. We are getting near it 👀

@lucca65
Copy link
Member

lucca65 commented May 13, 2022

ping @NeoVier, we should update this with master, it will get harder to do so as time progresses

@henriquecbuss
Copy link
Member Author

image

🚴

@lucca65
Copy link
Member

lucca65 commented May 13, 2022

damn... well we can simply drop this and make it again in the future.. by the looks we still got plenty to do before implementing this feature

@henriquecbuss
Copy link
Member Author

Actually the merge conflicts don't seem too bad to be honest

@netlify
Copy link

netlify bot commented May 14, 2022

Deploy Preview for cambiatus-elm-book failed.

Name Link
🔨 Latest commit 44501df
🔍 Latest deploy log https://app.netlify.com/sites/cambiatus-elm-book/deploys/627ef8127cfa590008e9c825

@henriquecbuss
Copy link
Member Author

@lucca65 I think we're probably not going to work on this in the foreseeable future (and haven't worked on it in a while). Should we close this PR?

Copy link
Member

@lucca65 lucca65 left a comment

Choose a reason for hiding this comment

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

Acho que o melhor a se fazer é alterar o PR para ser um PR que só coloca lib no projeto. Assim preservamos a excelente documentação e módulo que você fez, que facilita nosso trabalho mais tarde!

@henriquecbuss
Copy link
Member Author

Então bro, eu to pra lançar uma ferramenta que faz 90% do que esse PR faz (pega um ou mais contratos, e gera todo o código elm necessário pra interagir com eles). Se formos usar ela, não sei se vale muito a pena manter o que fizemos nesse PR - a maior parte do código desse PR teria que ser refeito.

Talvez podemos manter a branch pra manter a documentação, mas fechar o PR, o que acha?

Ainda não tá publicado no npm ou nos pacotes elm (mas já dá pra ir usando, tem um exemplo na pasta examples que faz o mesmo que o script pra pegar o saldo dos usuários), pq falta arrumar algumas coisas, mas a ferramenta é essa aqui: https://github.com/NeoVier/elm-eos. A ideia é ser bem parecida com dillonkearns/elm-graphql

@lucca65
Copy link
Member

lucca65 commented Sep 12, 2022

Nossa bro que show isso! beleza faz todo sentido então! parabéns pela visão, incrível ter algo desse tipo. Inclusive, uma recomendação pessoal que dou é de falar com o Luiz para ir atrás de um grant pra isso. É bem oq a pomelo e o eos foundation querem!

@henriquecbuss
Copy link
Member Author

Vou fechar o PR e manter a branch por enquanto. Vou fazer um port das funcionalidades de multisig que fizemos nesse PR como um exemplo no repo do elm-eos, e quando terminar, deleto a branch. Assim, se formos implementar multisig na Cambiatus, podemos seguir o exemplo de lá

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.

Add the ability to Multisig in our wallet
2 participants