-
Notifications
You must be signed in to change notification settings - Fork 275
fix: derive ata instead of creating a keypair for cpi and default state examples #935
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
fix: derive ata instead of creating a keypair for cpi and default state examples #935
Conversation
|
@datasalaryman is attempting to deploy a commit to the Solana Foundation Team on Vercel. A member of the Team first needs to authorize it. |
|
I missed the token 2022 references. Will work on them tomorrow |
8f81f2f to
0d7cbaa
Compare
|
this is ready for review. thanks |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
The existing Can you also update the other pages ( |
46c09b3 to
38cd2d6
Compare
38cd2d6 to
f996e90
Compare
|
@ZYJLiu - this pr will need another review. hopefully I didn't miss anything |
|
the snippets currently do not run on the vercel preview deployment with the changes. |
2992ad0 to
b5b7e23
Compare
|
@ZYJLiu - feel free to check now. I'm having a hard time testing it locally. Let me know if you have a way that works. |
|
the updated web3js on cpi guard page still not running |
are you updating these pages without checking the updated snippets work? |
|
hi @ZYJLiu I'm unable to on local because it takes awhile to run a dev server for this project on localhost. I also don't have access to the vercel project. I have a pesonal project I'm basing these examples on, but it isn't clear if it's translating one-for-one. Is there a way perhaps I can run a |
|
. |

Solana Docs recommend generating a keypair when creating a token account. This isn't standard practice and should be avoided.
All references to generating a keypair for the token account have been replaced with finding the token account with seeds.