feat(deposit-tracking): parse mint and burn event on solana #2137
Conversation
Signed-off-by: Gerhard Steenkamp <gerhard@umaproject.org>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
gsteenkamp89
requested review from
0xjorgen,
amateima,
dohaki,
mrice32 and
nicholaspai
as code owners
0xjorgen
changed the title
src/utils/cctp.ts
Outdated
|
|
||
| async queryMintAndWithdrawEvents(fromSlot?: bigint, toSlot?: bigint) { | ||
| return await this.queryEvents( | ||
| "MintAndWithdraw" as any, // Maybe override queryEvents method |
There was a problem hiding this comment.
Would the correct thing here be to add MintAndWithdraw to SVMEventNames in the SDK?
There was a problem hiding this comment.
This class is actually built for the SVM spoke contract. I am really just repurposing it's methods here since the event querying and unwrapping logic is the same as long as you initialize this with the correct program id and idl.
The best solution is probably to make changes to the original client in the sdk to infer event names based on the idl passed into its constructor
There was a problem hiding this comment.
this is removed anyway so no longer relevant f8e775f
src/utils/cctp.ts
Outdated
| return events.filter((event) => event.name === "DepositForBurn"); | ||
| } | ||
|
|
||
| async queryMintAndWithdrawEvents(fromSlot?: bigint, toSlot?: bigint) { |
There was a problem hiding this comment.
Are you using this method anywhere?
There was a problem hiding this comment.
No, I will remove it along with queryDepositForBurnEvents since those are not really viable options
| let fillEventTxRef = ""; | ||
|
|
||
| if (!fillEvent) { | ||
| if (bridgeProvider === "cctp") { |
There was a problem hiding this comment.
I dont uderstand why this is //todo?
Signed-off-by: Gerhard Steenkamp <gerhard@umaproject.org>
|
Deployment failed with the following error: Learn More: https://vercel.link/3Fpeeb1 |
Signed-off-by: Gerhard Steenkamp <gerhard@umaproject.org>
closes FE-217
Work done
CCTP (Sponsored)
CCTP (NOT Sponsored)