-
Notifications
You must be signed in to change notification settings - Fork 7
Transfer hook support #15
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
Conversation
9df4636
to
d68d4f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't look at the tests, but wanted to comment on the logic and setup of the repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! But do get an approval from Joe too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, left a few "take it or leave it" remarks. Nice work!!
"private": true, | ||
"scripts": { | ||
"programs:build": "zx ./scripts/rust/build-sbf.mjs program", | ||
"programs:build": "cargo-build-sbf", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yikes. I saw the discussion on this earlier, so I'm not going to make a stink here, but here's another situation where our CI is getting out of whack. Not your fault or within the scope of this PR, so you can disregard my griping. 🥴
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah... we should eventually pool all the updates to https://github.com/solana-program/create-solana-program and then update all the repos from that for consistency
If the mint has support for the TransferHooks extension, the program will now execute them upon a wrap or unwrap action.
Makes use of helpers exposed by token-2022 repo. However, there is a dependency on solana-program/token-2022#233 for multisig support.
New
test-transfer-hook
program created to test out new functionality.