-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: multiple executors #138
Conversation
@@ -4,8 +4,9 @@ | |||
"entryPoints": [ | |||
"0x5FF137D4b0FDCD49DcA30c7CF57E578a026d2789" | |||
], | |||
"relayer": "0xprivateKey", | |||
"beneficiary": "0x690b9a9e9aa1c9db991c7721a92d351db4fac990", | |||
"relayers": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we consider an option of using a seed phrase and specify the list of relayers as well?
if (privKey.startsWith("0x")) { | ||
wallets.push(new Wallet(privKey, provider)); | ||
} else { | ||
wallets.push(Wallet.fromMnemonic(privKey).connect(provider)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this mean we can also use a mnemonic? If so we need to document that in README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we can use mnemonics. Added it in README
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Please ask Taylor to update our documentation.
Description
relayers
config which accepts an array of private keys of relayersrelayer
config for backwards compatiblitybeneficiary
config optional, if it's not set, a relayer will always be a beneficiaryTested on Mumbai with 2 relayers and 2 userops at once
https://mumbai.polygonscan.com/tx/0xced5180faa7ec629a37bfd2b008a8d139f0112ecb3e777694a3f0eb58ee557f0
https://mumbai.polygonscan.com/tx/0xe230ab3fde49e0348f5bdc34dfa72e3458ae052926510dfb90bd6b97783d69d4
When sent another userop and all relayers are busy:
After some time, when one of the relayers is released:
Types of changes
What types of changes does your code introduce?
Put an
x
in the boxes that applyFurther comments (optional)