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

SafeL2 execTransactionFromModuleReturnData does not emit event #773

Closed akshay-ap closed 4 months ago

akshay-ap commented 4 months ago

Context/Issue

Recent changes in code which added Module guard feature resulted in SafeL2 contract not emitting SafeModuleTransaction in execTransactionFromModuleReturnData.

Expected outcome

execTransactionFromModuleReturnData in SafeL2 emits event SafeModuleTransaction

nlordell commented 4 months ago

Implemented in #772