Skip to content
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

Merged
merged 6 commits into from
Sep 7, 2021

Conversation

abarmat
Copy link
Contributor

@abarmat abarmat commented Jul 19, 2021

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.

@abarmat abarmat added the GIP label Jul 20, 2021
Copy link
Contributor

@davekaj davekaj left a 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 () {
Copy link
Contributor

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea

@abarmat abarmat force-pushed the ariel/gns-multicall branch from be047e6 to 94b8464 Compare August 9, 2021 13:39
@abarmat
Copy link
Contributor Author

abarmat commented Aug 9, 2021

Rebased

@nenadjaja
Copy link
Contributor

This is pretty cool @abarmat ! It will be easy to implement it on our end :-D

@abarmat abarmat added the enhancement New feature or request label Sep 7, 2021
@abarmat abarmat merged commit 329e92f into dev Sep 7, 2021
abarmat added a commit that referenced this pull request Sep 7, 2021
- Update license
- Allow batch transactions using multicall in the GNS (#485)
@abarmat abarmat deleted the ariel/gns-multicall branch September 22, 2021 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audited enhancement New feature or request GIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants