search

helius-labs / helius-sdk

123 stars 39 forks source link

feat: refactor smart transaction sender & confirmation #97

Closed kAsky53 closed 1 month ago

tavindev commented 1 month ago

What's missing to merge this? This might solve #112 with the new confirmation strategy.

0xIchigo commented 1 month ago

@tavindev this PR is currently stale and outdated with the latest changes to smart transactions. We might be better off closing the current PR and opening up a new one for this new confirmation strategy. Lmk if you're interested in either updating this PR or opening a new one!

tavindev commented 1 month ago

I went ahead and opened a new PR #114 @0xIchigo

0xIchigo commented 1 month ago

Going to close this in favor of #114