-
Notifications
You must be signed in to change notification settings - Fork 2
feat: sponsored burn ws sandbox #571
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
Conversation
| const depositArgs = decodeEventFromReceipt<DepositForBurnArgs>( | ||
| payload.transactionReceipt, | ||
| parseAbi(CCTP_DEPOSIT_FOR_BURN_ABI), | ||
| DEPOSIT_FOR_BURN_EVENT_NAME, | ||
| ); | ||
| if (depositArgs) { | ||
| args.destinationChainId = getCctpDestinationChainFromDomain( | ||
| depositArgs.destinationDomain, | ||
| ); | ||
| } else { |
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 thought that we're going the fetch events separately. OOC why is the destinationChainId needed to be associated with the SponsoredDepositForBurn? Is it for formatting addresses?
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.
Yes it is for formatting addresses, we do the same for the regular CCTP indexer here:
| const destinationChainId = getCctpDestinationChainFromDomain( |
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.
What do you mean with events being fetched separately?
The websocket listens to DepositForBurn and SponsoredDepositForBurn separately.
- DepositForBurn:
- SponsoredDepositForBurn: https://github.com/across-protocol/indexer/pull/580/files#diff-03ab2cf3bb27ec4c5e2aba5a25f4f038d51068d817e0bc976ac05c4a0455caaeR321
| * Stores a DepositForBurn event in the database. | ||
| * | ||
| * @param event The DepositForBurn entity to store. |
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.
SponsoredDepositForBurn
|
Closed in favour of #580 |
Adds Sponsored deposit for burn events to the websocket indexer.
The sponsored flow will be separated into three components:
Depends on #572