-
Notifications
You must be signed in to change notification settings - Fork 152
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
Allow batch transactions using multicall in the GNS #485
Conversation
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.
I am only requesting changes as I want to add some tests. I can add the tests. they are just an extra percaution I want to add
@@ -983,4 +983,68 @@ describe('GNS', () => { | |||
await gns.connect(me.signer).mintNSignal(me.address, 1, toGRT('10'), 0) | |||
}) | |||
}) | |||
|
|||
describe('batch calls', function () { |
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.
looks good. i would like to add a few tests to play it safe. i want to add the following sequences:
- publishSubgraph, publishSubgraph
- publishSubgraph, publishNewVersion
- publishSubgraph, mintNSignal, burnNsignal
I think it will be fine. I can write the tests for these
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.
Good idea
be047e6
to
94b8464
Compare
Rebased |
This is pretty cool @abarmat ! It will be easy to implement it on our end :-D |
Summary
By using a multicall a user can batch multiple operations into a single transaction.
An example of this is if the owner publishing a subgraph decides to mint some signal on that subgraph.