-
Notifications
You must be signed in to change notification settings - Fork 368
feat(ICRC-Rosetta): add ICRC-107 fee collector block handling #7697
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
base: master
Are you sure you want to change the base?
Conversation
…ciej-rosetta-feecol
| "#, | ||
| [], | ||
| )?; | ||
|
|
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.
With the metadata table, updating ckBTC balances take ~4m. This is similar to having an index on operation_type and searching for the fee collector in the blocks table and having a separate table for fee collectors.
| fee_collector: _, | ||
| caller: _, | ||
| } => ( | ||
| "107feecol", |
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 this constant should be defined in the icrc-ledger-types but it's not used there yet. I added a note to DEFI-1901 to create the constant when we implement the 107 schema.
Add fee collector block handling according to the ICRC-107 standard (https://github.com/dfinity/ICRC/blob/main/ICRCs/ICRC-107/ICRC-107.md). The fee collector information is stored in a separate table (
fee_collectors_107).