Should execute revert when success is 0?

[rdinicut]

There is no way to know if the transaction reverted when you use a client(Ex: web3) with a proxy identity to execute something.

Why not have something like:

function execute(uint256 _operationType, address _to, uint256 _value, bytes calldata _data)
        external
        onlyOwner
    {
        if (_operationType == OPERATION_CALL) {
            if(!executeCall(_to, _value, _data)) revert();
        } else if (_operationType == OPERATION_CREATE) {
            address newContract = executeCreate(_data);
            emit ContractCreated(newContract);
        } else {
            // We don't want to spend users gas if parametar is wrong
            revert();
        }
    } 

[frozeman]

The code changed, but i think a execute event could make sense.

[YamenMerhi]

Issue outdated