Skip to content
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

Define IDL for initialize permanent delegate instruction #44

Merged
merged 3 commits into from
Nov 13, 2024

Conversation

0xCipherCoder
Copy link
Contributor

This PR manually updates the Codama IDL to support all instructions in the permanent delegate instruction. This allows helpers to be generated for the new web3.js.
This pull request was created for https://app.gib.work/bounties/c17c023a-e6e0-4e4d-8dce-6e1a432f4c28 in an attempt to solve a bounty #26 . Payment for the bounty is immediately sent to the contributor after merge.

Copy link
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good. Just one little change and we should be good to merge.

Comment on lines 24 to 29
const mintAddress = await createMint({
authority,
client,
extensions: [permanentDelegateExtension],
payer: authority,
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because here, we're testing the initialisation, would you mind using the longer version instead of the createMint helper?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure will update this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lorisleiva I have updated code with a longer version of mint and rebased it with the main branch. Please review.

Copy link
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect 👌

@lorisleiva lorisleiva merged commit 0c8b0b8 into solana-program:main Nov 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants