aragon / client

(Aragon 1) Create and manage decentralized organizations on Ethereum.
https://client.aragon.org
GNU Affero General Public License v3.0
829 stars 273 forks source link

Agent, Finance: show direct token/eth transfers #776

Open lkngtn opened 5 years ago

lkngtn commented 5 years ago

Is your feature request related to a problem? Please describe. If you want to use a vault/agent instance to hold funds, and the finance app to view the current state of assets in a vault, there can be discrepancy between the amount of funds in the vault and the transaction log in the finance app. This can happen with the agent app if you make token transfers directly with the dao act command. Or when multiple applications (such as TPS projects or allocations) are connected to the same vault as the finance app.

Describe the solution you'd like The transaction log displayed in finance should show all incoming and outgoing transfers for watched tokens in its log, even if they were not triggered via the finance app directly.

Describe alternatives you've considered If all transactions were routed through the finance app (including transfer requests associated with other apps like projects or allocations) then finances transaction log (and also its budgeting capability) would work correctly. However, this seems problematic unless we created a tight coupling between the vault/agent/finance app functionality.

Additional context This same issue also breaks the finance's hidden budget functionality, as the budget only applies to transactions which are sourced from the finance app, and not from other sources.

sohkai commented 5 years ago

Related: https://github.com/aragonone/product/issues/51

Incoming transfers from the vault should not be a problem to visualize.

The budgeting problem can be mitigated slightly if we're able to provide some documentation for the Vault and Agent's transfer roles to be primarily given to an attached Finance app, and that direct transfers should happen through the Finance app. With the "safe" execution improvements to the Agent app, typical use should consist of non-financial interactions with other contracts so that financial ones happen with the Finance app.

However, this does start to make a organization-level constraints module sound more and more compelling.

luisivan commented 5 years ago

This item will be shortly put into one of A1's sprints for the upcoming releases

marsrobertson commented 4 years ago

I support this initiative.

https://aragon.chat/channel/dev-help?msg=xWKude6xscoLKxY6J

(great experience, of course some rough edges here or there but overall using Aragon is like music)

lkngtn commented 4 years ago

Bumping this as well as it is quite common to receive funds directly to the agent and it can be very confusing to see a partial history.