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

Draft: Support permissioned candymachines - Basic #47

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dankelleher
Copy link

Demonstrate how to integrate a UI with a permissioned candymachine using this fork.

TODO before merge - more testing to ensure backwards compatibility. Until the metaplex fork is merged, have a switch on the program ID, so that gated and non-gated candymachines are supported

@kevinfaveri
Copy link
Owner

What is the diff between this one and this: #46

Btw, I will wait for the metaplex fork to be merged before taking action on this one too, but it seems a nice addition!

@dankelleher
Copy link
Author

Thanks! Yes that makes sense. I will also definitely be adding more docs and explanations before expecting you to merge. :)

#46 includes a UI widget from Civic which takes users through a KYC process before granting access to the candymachine.
This one is a simpler and "purer" integration that just checks if the wallet has access or not, no widget.

@kevinfaveri
Copy link
Owner

kevinfaveri commented Oct 23, 2021

@dankelleher There are now conflicts because I updated the metaplex/js library.

Also, I agree. Without a widget is better.

@kevinfaveri kevinfaveri mentioned this pull request Oct 28, 2021

export const CANDY_MACHINE_PROGRAM = new anchor.web3.PublicKey(
"cndyAnrLdpjq1Ssp1z8xxDsB8dxe7u4HL5Nxi2K5WXZ"
// "cndyAnrLdpjq1Ssp1z8xxDsB8dxe7u4HL5Nxi2K5WXZ"

Choose a reason for hiding this comment

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

Commented code should not be pushed to GitHub.

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.

3 participants