🍉 - Add Token 2022 Basics Program #22
Conversation
| }); | ||
|
|
||
| /* | ||
| // Comment out because we use init in the transfer instruction |
There was a problem hiding this comment.
Do we need to include the "create_associated_token_account" instruction then at all?
There was a problem hiding this comment.
As an example reference, yes. The payer's ATA in the tests are also created with this instruction.
There was a problem hiding this comment.
Is the process for creating an ATA for a Token2022 mint different than legacy mints?
There was a problem hiding this comment.
nvm, I see where the reference is needed.
Yeah, good with me, leave it in. Just put a little message in your comment header that mentions that you're including this instruction as a reference, but it's not required for the test.
The message you have there isn't super clear at first why it's commented out
There was a problem hiding this comment.
Something like "This instruction is included only as a reference, but is not required to run this test, because we are using init in the transfer instruction. The create_associated_token_account instruction on the program is provided as a reference as well"
There was a problem hiding this comment.
Sounds good, will make these changes.
There was a problem hiding this comment.
Changes made, ready to merge.
2 participants
No description provided.