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

Identifier 'mintToken' has already been declared. (211:8) #26

Open
vvronskyFX opened this issue Jan 30, 2022 · 6 comments
Open

Identifier 'mintToken' has already been declared. (211:8) #26

vvronskyFX opened this issue Jan 30, 2022 · 6 comments

Comments

@vvronskyFX
Copy link

I've run your code to see the difference between what you have and what BuildSpace has. When running it, I can across this error "Identifier 'mintToken' has already been declared. (211:8)"

Looks like you have two "const mintToken ...." in the candyMachine > index.js file.

Did you manage to get this project running on your end with a full functioning UI?

@MikeDupree
Copy link

@vvronskyFX I think it's due to a bad merge.

Looking at 672a7ce

You can see he removed the first mintToken function.

But in main they both exist.

I'm working through it now if I get it working Ill post an update.

@MikeDupree
Copy link

The creator's fork is in a more working condition then it is in the buildspace codebase

See: https://github.com/AlmostEfficient/CandyMachineV2

Late night trying to create a PR I think he made a few mistakes. It happens.

@noahh40
Copy link

noahh40 commented Feb 4, 2022

any update? @dooptroop I am currently stuck on this.

@MikeDupree
Copy link

MikeDupree commented Feb 4, 2022

any update? @dooptroop I am currently stuck on this.

I have it fixed in my project. I could look to post a PR this weekend if I get sometime.

If your stuck tho review 672a7ce and even https://github.com/AlmostEfficient/CandyMachineV2

One of the mintToken methods needs to be removed.

@MikeDupree
Copy link

MikeDupree commented Feb 4, 2022

@noah40povis This PR looks to be addressing the issue #27

Removing the following mint function b0dc1ac

@AlmostEfficient
Copy link
Contributor

Sorry folks, not sure what happened in a previous merge, looks like a merge conflict gone wrong. #27 looks good, I've asked the team to merge.

Thank you!

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

No branches or pull requests

4 participants