dao-mix icon indicating copy to clipboard operation
dao-mix copied to clipboard

Feature/multi call roles

Open azizazlan opened this issue 2 years ago • 6 comments

Now with multicall to grant and revoke roles.

azizazlan avatar Mar 18 '22 11:03 azizazlan

Love the suggestion here! Thank you!

But could we fix the PR so it's easier to review? Most of the changes look like they could be in a .gitignore

PatrickAlphaC avatar Mar 19 '22 03:03 PatrickAlphaC

OK I have fixed the formatting -now with 4 spaces tab width. (I was using 2 spaces tab width).

Sorry I am trying to understand your comment: Most of the changes look like they could be in a .gitignore

Anyway I added artifacts and [._]*.s[a-v][a-z] in the file because I am using vim as the editor.

Let me know.

azizazlan avatar Mar 19 '22 10:03 azizazlan

Right. Sorry it's been hard to see what your changes are doing.

I think this one too is just looking to implement this with hardhat? Could you check this repo and see if it has everything you need?

https://github.com/PatrickAlphaC/dao-template

PatrickAlphaC avatar Mar 21 '22 11:03 PatrickAlphaC

Thank you for replying my request. I will check it out and let you know.

On Mon, 21 Mar 2022 at 7:52 PM, Patrick Collins @.***> wrote:

Right. Sorry it's been hard to see what your changes are doing.

I think this one too is just looking to implement this with hardhat? Could you check this repo and see if it has everything you need?

https://github.com/PatrickAlphaC/dao-template

— Reply to this email directly, view it on GitHub https://github.com/brownie-mix/dao-mix/pull/4#issuecomment-1073802339, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFSXJ6EKEGAMDWAREUV4QLVBBPIBANCNFSM5RBR2YWQ . You are receiving this because you authored the thread.Message ID: @.***>

azizazlan avatar Mar 23 '22 06:03 azizazlan

Below are your comments in the script file deploy_and_run.py

    # Now, we set the roles...
    # Multicall would be great here ;)

So, I was trying to do multicall to set the roles and at the same time do it hardhat task.

What I should have done is implement it in brownie/python script - this would be a clearer "pull request".

I learnt my mistake and sorry I make you confuse.

azizazlan avatar Mar 24 '22 02:03 azizazlan

Ahhhhh, yes exactly!

PatrickAlphaC avatar Mar 28 '22 18:03 PatrickAlphaC