search

safe-global / safe-smart-account

Safe allows secure management of blockchain assets.
https://safe.global
GNU Lesser General Public License v3.0
1.89k stars 929 forks source link

Feature request: Transaction meta data #331

Open tschubotz opened 3 years ago

tschubotz commented 3 years ago

Context / issue

Transaction meta data: I would like to be a to (optionally) provide meta data when calling execTransaction.

Why? Safe transactions are triggered via different interfaces. Also the officiel safe-react interface has Safe apps. The information on which txs was triggered via which Safe app is currently stored exclusively in our backend. This is not great for analytics since we can't easily match it with on-chain data from dune.

Proposed solution

Add an optional field to execTransaction where arbitrary data could be provided. This data should not be used at all for tx execution. It's sole purpose is for this info to be stored on chain via the call trace.

Alternatives

Additionally trigger an event with this info. I currently don't think this is necessary.

mmv08 commented 3 years ago

Maybe this can be useful: https://github.com/0xsequence/sstore2

Slyracoon23 commented 1 year ago

I am actually interested in this feature as well. Especially for DAO that use Gnosis-safe for distributing funds.