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

token: Add extension support in mint init #2744

Merged
merged 1 commit into from Jan 19, 2022

Conversation

joncinque
Copy link
Contributor

@joncinque joncinque commented Jan 15, 2022

Problem

People want to initialize mints with extensions

Solution

Add that ability. A few notes from this PR:

  • introduces StateWithExtensionsOwned, used in testing and client situations, where the TLV data is owned instead of borrowed. There's some copy-pasta because vec uses split_off and slice uses split_at
  • can initialize a mint close authority
  • initialize mint supports extensions
  • ExtensionInitializationParams for cleaner init!

Edit: Reduced the scope of this to just initialization and tests. The actual support for the mint close authority will come in a separate PR

@joncinque joncinque changed the title token: Add mint close authority support token: Add extension support in mint init Jan 18, 2022
@joncinque joncinque marked this pull request as ready for review January 18, 2022 20:29
@joncinque joncinque requested a review from mvines January 18, 2022 20:29
@joncinque joncinque added this to In progress in SPL Token Extensions Jan 18, 2022
@joncinque joncinque moved this from In progress to Done in SPL Token Extensions Jan 18, 2022
@joncinque joncinque moved this from Done to In progress in SPL Token Extensions Jan 18, 2022
@joncinque joncinque removed this from In progress in SPL Token Extensions Jan 18, 2022
@joncinque joncinque merged commit 480dc68 into solana-labs:master Jan 19, 2022
@joncinque joncinque deleted the tk-close-mint branch January 19, 2022 11:44
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.

None yet

2 participants