-
Notifications
You must be signed in to change notification settings - Fork 0
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
Support nethermind plugin as data source #15
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.
Mostly FIPF, a little bit LGTM - because im very new to this code
@@ -4,7 +4,7 @@ namespace CirclesUBI.PathfinderUpdater; | |||
|
|||
public static class Block | |||
{ | |||
public static async Task<long> FindByTransactionHash(string connectionString, string transactionHash, Queries queries) | |||
public static async Task<long> FindLatestBlockNumber(string connectionString, Queries queries) |
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.
just curious how the function name and signature changed but nothing else in the 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.
I think i understand how it worked before, and see that now you want to go over every block?
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 catch. That's basically dead code that was once needed to do partial updates to the pathfinder.
@@ -70,45 +70,46 @@ from cache_crc_balances_by_safe_and_token | |||
order by ""blockNumber"" desc, ""transactionIndex"" desc, ""logIndex"" desc; | |||
"; | |||
|
|||
// TODO: Use the Discounted event from 0.3.4 to get precise demurraged balances |
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.
should we already store "when" (on what day) we've queried these balances?
No description provided.