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 tracking of pools #492

Merged
merged 11 commits into from
Aug 1, 2022
Merged

Remove tracking of pools #492

merged 11 commits into from
Aug 1, 2022

Conversation

jamiehewitt15
Copy link
Member

@jamiehewitt15 jamiehewitt15 commented Jul 20, 2022

Fixes #492

Changes proposed in this PR:

  • Removing all mappings related to pools
  • Removing pools from the schema and the subgraph template

@jamiehewitt15 jamiehewitt15 changed the title Issue 491 remove pools Remove tracking of pools Jul 20, 2022
@jamiehewitt15 jamiehewitt15 marked this pull request as ready for review July 20, 2022 14:46
@mihaisc
Copy link
Contributor

mihaisc commented Jul 27, 2022

Merged main with various dep updates. Because these update caused #505 I decided to fix the issues here because i didn't want to spend more time fixing these issues for pools as well.
Also fixed a bug where totalFixedSwapVolume was incorrect , it was adding datatokens instead of base tokens.
Did a partial sync on rinkeby with no errors.

@mihaisc mihaisc requested a review from alexcos20 August 1, 2022 11:57
@mihaisc mihaisc merged commit 1289a08 into main Aug 1, 2022
@mihaisc mihaisc deleted the issue-491-remove-pools branch August 1, 2022 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants