safe-global / safe-smart-account

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

Index essential addresses when emitting events #541

Closed mmv08 closed 1 year ago

mmv08 commented 1 year ago

Context/issue

Per suggestion from the safe forum, https://forum.gnosis-safe.io/t/safe-contract-v1-4-0/2109/20, it's tricky to index module changes off-chain.

Proposed solution

Take the suggestion and widen the scope a little bit, and index all essential addresses in the safe contract:

Alternatives

Additional context