search

gnosisguild / zodiac-safe-app

https://gnosis-safe.io/app/share/safe-app?appUrl=https://zodiac.gnosisguild.org/&chainId=5
GNU Lesser General Public License v3.0
26 stars 28 forks source link

Remove module cannot be added to transaction bundle #15

Closed auryn-macmillan closed 3 years ago

auryn-macmillan commented 3 years ago

"Remove module" cannot be added to transaction bundle.

carlosfebres commented 3 years ago

I think we can always add the remove module transaction to the transaction builder. I made a screen record to illustrate the new flow of removing a module. (the tx builder opens after adding the tx) Would love to heard your feedback on this @auryn-macmillan @samepant

https://user-images.githubusercontent.com/2939980/130914055-6e49740b-c378-4a22-8a7a-3dc3a65734e9.mp4

samepant commented 3 years ago

This looks good to me