aragon / aragon-cli

CLI for creating and publishing Aragon apps
GNU General Public License v3.0
91 stars 79 forks source link

'dao install' sent a transaction which doesn't install an app #1764

Open ethene opened 4 years ago

ethene commented 4 years ago

πŸ› Bug Report

After executing 'dao install' the tool has executed a transaction which doesn't end up with the app being installed

Have you read the Contributing Guidelines on issues?

To Reproduce

aragon dao install stablefund.aragonid.eth token-request.aragonpm.eth --app-init-args 0xe8aa182d10252bed21cbe35a1f8cac9b0d9458a7 0xc1ee97ce717c1dfed21543ca56f23303c1d5dffc "[]" --environment aragon:mainnet --gas-price 37 βœ” Fetching token-request.aragonpm.eth@latest βœ” Fetching token-request.aragonpm.eth@latest ↓ Checking installed version [skipped] β†’ Installing the first instance of token-request.aragonpm.eth in DAO βœ” Deploying app instance ↓ Fetching deployed app [skipped] β†’ App wasn't deployed in transaction.

β„Ή Successfully executed: "Execute desired action as a token holder"

TX: https://etherscan.io/tx/0x11911c9abc21ab2f9545088276fcfbca6a237a3328a377e335d3ddc105194092

App is not seen in the configuration, on UI or 'aragon dao apps'

Expected behavior

I expect the cli to install the new app which can be seen and configured

Actual Behavior

TX created: https://etherscan.io/tx/0x11911c9abc21ab2f9545088276fcfbca6a237a3328a377e335d3ddc105194092

app not seen anywhere in the configuration

Context

mainnet

Organization

stablefund.aragonid.eth

Environment

MacOS Mojave CLI from NPM

0xGabi commented 4 years ago

Hi @ethene this an expected behavior. If you look into the success message: Execute desired action as a token holder what the aragonCLI did was trying to install the app but as you don't have the full permissions to do it directly it creates a Vote that if gets approve the app will be installed. Looking into your organization, this is the open vote: https://mainnet.aragon.org/#/stablefund/0x241b27133d78177f1fb37ea26645bff0d1e1bbad/vote/0/

If you are the only owner fo the organization and you would like to do a quick setup without having to vote on every change, you can assign your account the role to create permissions: https://mainnet.aragon.org/#/stablefund/permissions/app/0xe0f35ac8e6c8b8ab88d4798becc0865496d623ba

The default configuration of most Aragon Organization Templates has a Voting app as the most privileged.

ethene commented 4 years ago

Thanks for the explanation. Now that the vote has been approved what next steps needs to be performed to install the app? I still don't see it in the configuration.

sohkai commented 4 years ago

@ethene You just need to assign the app a permission!

ethene commented 4 years ago

@sohkai app is not seen in the configuration yet, it is not possible to assign any permissions to it.

I approved a vote per @0xGabi suggestion, the app did not appear.

Then I added my account a role to create permissions per this suggestion too.

After executing the 'install' command again I see the following:

aragon dao install stablefund.aragonid.eth token-request.aragonpm.eth --app-init-args 0xe8aa182d10252bed21cbe35a1f8cac9b0d9458a7 0xc1ee97ce717c1dfed21543ca56f23303c1d5dffc "[]" --environment aragon:mainnet --gas-price 33 βœ” Fetching token-request.aragonpm.eth@latest βœ” Fetching token-request.aragonpm.eth@latest βœ” Checking installed version βœ” Deploying app instance ↓ Fetching deployed app [skipped] β†’ App wasn't deployed in transaction.

β„Ή Successfully executed: "Execute desired action as a token holder"

I see another vote created and the app still does not exist in the configuration. Really disappointed there is no proper documentation on the app installations at all

sohkai commented 4 years ago

Thanks for the feedback @ethene, we are equally as disappointed in ourselves.

Your app is installedβ€”it's just a bit of an awkward path to find its address so you can assign a permission to it as you weren't able to complete the interaction through the CLI.

0xc0af05fdf7e8338ee5359b43a1c69a3261153ba6 is your new app's address!

ethene commented 4 years ago

What can I do with the address to link it to the DAO app center to enable it? I noticed there is no choice in the UI and what cli commands it do I need to perform for it?