Armani Ferrante
Armani Ferrante
This sounds awesome. I think if this is done, we should make a separate program (and thus a separate repo) for it, unless there's a good reason to put it...
Great point. We can probably just add an additional method for performing both the `set_owners` and `change_threshold` operation together. What do you think? Also, if the runtime maintains the PDA...
@ruuda take a look at https://github.com/project-serum/multisig/pull/5. It should address this use case. If not let me know.
Also, I think I'd prefer to keep this multisig as simple as possible, so I'd like to avoid allowing multiple instructions in the transaction data structure. If one needs multiple...
Let's leave the separate ones there for now. Removing them would require some (quite minor) UI updates.
The main reason to not garbage collect the account is that it makes it more difficult for UIs to display the data.
Can we use https://github.com/solana-labs/wallet-adapter instead? It should eliminate a lot of boilerplate.
The app in `example/` is for educational purposes and so there is no plans to host it for mainnet. Perhaps this can be used with devnet, however.
0x66 is the zero swap error, meaning the size of the trade was probably too low. See https://github.com/project-serum/swap/blob/master/programs/swap/src/lib.rs#L723. Did you try swapping for more?
Additionally, we should disable the swap button if the token quantity is too small.