Skip to content

Conversation

@stegaBOB
Copy link
Contributor

Uses the UUID as feature flags. Checks byte offsets to determine if a feature is active. Implemented working collections flag as POC.

@roederw roederw self-assigned this Mar 29, 2022
@roederw
Copy link
Contributor

roederw commented Mar 29, 2022

I like this change. @austbot @bhgames there is no currently intended use for the UUID moving forward in CMv2, right?

@bhgames
Copy link
Contributor

bhgames commented Mar 30, 2022 via email

Copy link
Contributor

@austbot austbot left a comment

Choose a reason for hiding this comment

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

Discussion pending

@austbot
Copy link
Contributor

austbot commented Apr 7, 2022

As per our Discussion @stegaBOB lets move this feature to have a version in the UUID:
Do a bit of analysis to make sure our new version wont get tipped by an existing uuid: you will need a delimiter before the version byte to ward off any collision.

Then in the program use a version to determine what version of the config account to use.
You can get fancy with traits and make this nice and then all the candy machine configuration will be in the config account.

This can actually made the collection config pda irrelevant. In general great idea to put the uuid to good use.

@austbot
Copy link
Contributor

austbot commented Apr 25, 2022

@stegaBOB how is the new way going?

@stegaBOB
Copy link
Contributor Author

stegaBOB commented May 5, 2022

Closing this PR in favor of #418

@stegaBOB stegaBOB closed this May 5, 2022
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.

4 participants