Delete owner’s delegated keys when the owner is removed

[katspaugh]

A feature request from the Yearn team:

If an owner who an added a delegate key has been removed, that delegate is no longer valid. So it would be good to remove all delegates by that owner when the owner is removed/replaced.

[schmanu]

I think this has to be fixed in the transaction service. Only the delegator or delegate can remove a delegate.. If the delegator is removed as an owner by someone else, this other owner can not sign the remove delegate payload.

Our indexer should pick up that an owner got removed and delete all delegates where this owner was the delegator.

[katspaugh]

Makes sense. @Uxio0 wdyt?

[Uxio0]

Makes sense @katspaugh , could you open an issue on the tx service?

[katspaugh]

@Uxio0 Please feel free to transfer this one, I don't have the right permissions for that repo. I gave you admin rights in this one.

[Uxio0]

I don't have permissions to transfer this issue 😅

[Uxio0]

@moisses89 Delegates should only be deleted for Delegates for that owner if the Safe is set. For delegates valid across multiple Safes it shouldn't be deleted