Uh oh!
There was an error while loading. Please reload this page.
Fix default Solana token list and support transfer (not just transferChecked) - #1286
Conversation
337a24f to
c43b8feCompare
danrmiller
left a comment
There was a problem hiding this comment.
I think the token transaction parsing is missing the "innerInstructions" and those get missed.
I'm also not sure about the splTransfers.length != 1, I'm pretty sure that is going to miss things on multi-recipient transactions.
But those can be addresses seperately.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@## staging #1286 +/- ##
========================================
Coverage 74.89% 74.89% ========================================
Files 11 11 Lines 685 685 ========================================
Hits 513 513 Misses 172 172 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
a8726af to
c43b8feCompareUh oh!
There was an error while loading. Please reload this page.
No description provided.