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

Remove AllocationExchange.claimMany() #775

Closed
alex-pakalniskis opened this issue Sep 15, 2023 · 0 comments · Fixed by #779 or #774
Closed

Remove AllocationExchange.claimMany() #775

alex-pakalniskis opened this issue Sep 15, 2023 · 0 comments · Fixed by #779 or #774
Labels
enhancement New feature or request indexer-agent

Comments

@alex-pakalniskis
Copy link
Contributor

alex-pakalniskis commented Sep 15, 2023

Remove AllocationExchange.claimMany() to simplify indexer-agent logs

See Discord:

With the new contracts indexer-agent only needs to call AllocationExchange.redeemMany() no more claimMany().
Now that the protocol upgrade is complete we can remove it completely
-Ford

@alex-pakalniskis alex-pakalniskis converted this from a draft issue Sep 15, 2023
@alex-pakalniskis alex-pakalniskis changed the title Remove claimMany() from indexer-agent Remove AllocationExchange .claimMany() Sep 15, 2023
@alex-pakalniskis alex-pakalniskis changed the title Remove AllocationExchange .claimMany() Remove AllocationExchange.claimMany() Sep 15, 2023
@alex-pakalniskis alex-pakalniskis added the enhancement New feature or request label Sep 15, 2023
@fordN fordN moved this from 🗃️ Inbox to 🚩 In Review in Indexer Sep 15, 2023
@github-project-automation github-project-automation bot moved this from 🚩 In Review to ❌ Closed in Indexer Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request indexer-agent
Projects
Status: Closed
1 participant