Fungible Assets Deposit and Withdrawal Events

[leosupra]

If possible, there’s a small improvement that could be very valuable.

The events for fungible asset deposits and withdrawals :

0x1::fungible_asset::Deposit
0x1::fungible_asset::Withdraw

could include the fungible asset contract address, as well as the sender/receiver account.

This would make it easier to monitor on-chain fungible asset activity.

[sjoshisupra]

Deposit event contains the store_addr, from there, it is possible to lookup metadata, which is essentially FA address. Also, store_addr itself is a store object on which one can invoke root_owner to find the owner.

[leosupra]

Deposit event contains the store_addr, from there, it is possible to lookup metadata, which is essentially FA address. Also, store_addr itself is a store object on which one can invoke root_owner to find the owner.

That requires two calls to fetch the data, which isn’t scalable. In contrast, legacy coin deposit and withdrawal events already include the contract address and the account.