project-serum / serum-ts

Project Serum TypeScript monorepo
https://projectserum.com
Apache License 2.0
273 stars 246 forks source link

Replace `Account` to `Keypair` #99

Open kazuki-hanai opened 3 years ago

kazuki-hanai commented 3 years ago

Since v1.10.0 at @solana/web3.js, Account was deprecated. So we should update @solana/web3.js package and replace Account with KeyPair.

armaniferrante commented 3 years ago

This is news to me. An excellent suggestion.

johnsonchak commented 3 years ago

@armaniferrante but in the demo, market.placeOrder() still accept owern as an Account Type. What should i do with that? plz help~

Gsuz commented 3 years ago

the code needs to be refactored.

ShaneBurkhart commented 3 years ago

Are all market.ts calls with Account broken? I'm getting 410 Gone for all calls that include an Account. But I can pull bid and ask orders just fine.

Any ideas?

ivanbara commented 2 years ago

Please provide a working example code since the one in README is outdated after this change.