Enable metatransactions throughout the contract

[cinnabarhorse]

Matic supports meta transactions (gasless transactions) and suggests we use Biconomy as a service provider. We will need to make some small changes to our smart contracts to enable these.

https://docs.biconomy.io/tutorials/native-meta-transactions

https://github.com/bcnmy/metatx-standard

[cinnabarhorse]

We can start by making a small demo contract and deploying to Mumbai with a demo UI so users can start experiencing metatransactions on Matic.

[cinnabarhorse]

@mudgen anything left to add for this? If not I will close.

[mudgen]

I have modified the metatransactions in the contract to work so that Ethereum can be connected in metamask but the metatransactions can be for Matic Network. But if we aren't going to do this, and I don't think we should, then I think I should change metatransactions back to the way they were before because I think that's more standard.

[cinnabarhorse]

Yes I agree, let's change them back to the original configuration.

[mudgen]

I think this is the only changes I need to make to the contract:

bytes32 internal constant EIP712_DOMAIN_TYPEHASH =
        keccak256(bytes("EIP712Domain(string name,string version,uint256 chainId,address verifyingContract)"));

function domainSeparator(string memory name, string memory version) internal view returns (bytes32 domainSeparator_) {
  domainSeparator_ = keccak256(
  abi.encode(EIP712_DOMAIN_TYPEHASH, keccak256(bytes(name)), keccak256(bytes(version)), bytes32(getChainID()), address(this))
  );
}

I think you will have to make a related change in the UI.

It is done in the contract.

Commit: a1402828ed9cbea795b92c76aa28f53cae5afddf

[mudgen]

This is done in AavegotchiDiamond -- need to do it for GHSTStakingDiamond

[cinnabarhorse]

Completed.