aragon / aragon-cli

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

Permissions are not setup correctly for the Counter app(?) #214

Closed kernelwhisperer closed 5 years ago

kernelwhisperer commented 5 years ago

To reproduce this:

  1. aragon init foo
  2. aragon run (pin parcel-bundler to v1.9.3 if running fails)
  3. Press increment in the counter app

image

Browser console logs: transaction bag {transaction: undefined, path: Array(0), accept: ƒ, reject: ƒ}

  1. Add a new permission image

What happens next is that the permission is granted and the confirmation(?) fails: image

  1. Trying to increment gives the same error

Note:

Environment: aragon-cli version 5.0.0

Any thoughts on how to debug this?

izqui commented 5 years ago

@0x6431346e Just to confirm, the network and the account are properly set in Metamask right? The first address that gets printed when doing aragon run or aragon devchain should be allowed to perform the action and is the manager for said permission.

Running dao acl [dao-address] throws Cannot find module '@aragon/apps-token-manager/arapp'

That's quite weird, can you try using the latest master for the CLI and seeing if the error persists? We recently changed how the roles of known apps get calculated in #209 (still unreleased)

cjyabraham commented 5 years ago

I got the same issue and yes, I've set the Metamask account correctly I think.

kernelwhisperer commented 5 years ago

@izqui this was resolved by deleting .aragon/ganache-db-8545

Maybe we should consider having a different ganache folder for each aragon os version ? :thinking:

luisivan commented 5 years ago

That'd make a lot of sense @0x6431346e

sohkai commented 5 years ago

@0x6431346e Should we close this for #137?

kernelwhisperer commented 5 years ago

Yep