Skip to content

Create and save missing accounts in cache, enable Sollet unwrap#77

Open
secretshardul wants to merge 2 commits intoproject-serum:armani/wrapfrom
cykura:wrap
Open

Create and save missing accounts in cache, enable Sollet unwrap#77
secretshardul wants to merge 2 commits intoproject-serum:armani/wrapfrom
cykura:wrap

Conversation

@secretshardul
Copy link
Contributor

@secretshardul secretshardul commented Sep 1, 2021

Extends upon #64 . Saves created OpenOrders and associated token accounts in cache, allowing swaps to be performed for new tokens.

As a design change, a "no balance" message is shown if associated token account for toMint is not present. If a toWallet is not present, logically users will not swap tokens out of it.

Sollet unwrap for wUSDC and wUSDT is now supported.

@secretshardul secretshardul force-pushed the wrap branch 4 times, most recently from 6ca1cb7 to a3b04e4 Compare September 2, 2021 10:54
@secretshardul secretshardul changed the title WIP: Save created OpenOrders and token accounts in state Create and save missing accounts in cache Sep 2, 2021
@secretshardul
Copy link
Contributor Author

secretshardul commented Sep 11, 2021

It would be great if you could review and merge #64 and #77 in master. We've already deployed this branch in production. Wrap/unwrap for Sollet is now supported.

@secretshardul secretshardul force-pushed the wrap branch 2 times, most recently from 5c12f3b to caa3f08 Compare September 12, 2021 13:02
@secretshardul
Copy link
Contributor Author

A bug in the code was squashed. The old code to add a new token account in cache was a stub.

Allow swaps to be performed after necessary accounts for 'toMint'
and 'quoteMint' are created.
@secretshardul secretshardul changed the title Create and save missing accounts in cache Create and save missing accounts in cache, enable Sollet unwrap Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant