-
-
Notifications
You must be signed in to change notification settings - Fork 515
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
Utilize TheGraph for Aave. Also support Aave borrowing. #1545
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1545 +/- ##
===========================================
- Coverage 71.49% 71.37% -0.12%
===========================================
Files 359 363 +4
Lines 19527 20027 +500
Branches 2778 2862 +84
===========================================
+ Hits 13960 14294 +334
- Misses 4841 4968 +127
- Partials 726 765 +39
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
e02808f
to
05099dc
Compare
This pull request introduces 3 alerts when merging e90627298cdc9e60df119d0ec4643cf85202c967 into 239552b - view on LGTM.com new alerts:
|
Split into blockchain/graph history inquiring modules
Essentially offer almost the same functionality we have via normal blockchain querying but from the graph
e906272
to
96118f6
Compare
Fix #1458. Should also add Aave borrowing events.