Fix Chainlight 005 // Introduce withdrawal delay and min deposit

[filmakarov]

Fix this.

• Introduce submit request / execute request pattern for paymaster id balances withdrawals. Delay can be set to 0 for trusted paymasterIds though

• introduce min deposit.

• Introduce code changes

• Fix tests

[livingrockrises]

let's merge below PR to develop first, take back merge and work on the test cases
https://github.com/bcnmy/gasdaddy/pull/28/files

[livingrockrises]

let's merge below PR to develop first, take back merge and work on the test cases
https://github.com/bcnmy/gasdaddy/pull/28/files

[filmakarov]

let's merge below PR to develop first, take back merge and work on the test cases https://github.com/bcnmy/gasdaddy/pull/28/files

looking at it

[wooz3k-t]

Here's a minor suggestion.

• Emit an event in setTrustedPaymasterId(), setMinDeposit(), and withdrawEth().
• Add require(_trustedPaymasterIds[paymasterId] != isTrusted) in setTrustedPaymasterId().

[livingrockrises]

Here's a minor suggestion.

• Emit an event in setTrustedPaymasterId(), setMinDeposit(), and withdrawEth().
• Add require(_trustedPaymasterIds[paymasterId] != isTrusted) in setTrustedPaymasterId().

we thought these actions may not need event tracking. can still add @filmakarov

[filmakarov]

Here's a minor suggestion.

• Emit an event in setTrustedPaymasterId(), setMinDeposit(), and withdrawEth().
• Add require(_trustedPaymasterIds[paymasterId] != isTrusted) in setTrustedPaymasterId().

addressed here 36fb443