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

fix: metadata image correctly replaced with consecutive number so it works with cmv2 #11

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

Conversation

nukec
Copy link

@nukec nukec commented Dec 25, 2021

New candy machine v2, requires metadata image property to be consecutive number of asset, as image.png doesn't work anymore, and doesn't replace. Change was made that .json file is prepared with consecutive number for image property for instance 0.png for 0.json, 1.ong for 1.json. etc

@nukec nukec changed the title fix: metadata image correctly replaced with consecutive number fix: metadata image correctly replaced with consecutive number so it works with cmv2 Dec 25, 2021
@t-araujo
Copy link

Thanks @nukec, lets put this to main please

Copy link
Contributor

@ck256-2000 ck256-2000 left a comment

Choose a reason for hiding this comment

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

This doesn't work as intended. it will retitle and replace image.png with a 1.png, 2.png, etc, but nftsCreatedCounter begins at 1 so you'll always be off by one. Ask me how I know :) I've submitted a fix that occurs in nft.py, last step before writing the JSON file.

@Solseum
Copy link
Owner

Solseum commented Jan 19, 2022

It doesnt matter atm if is image.png or the number.png both works on metaplex upload using the latest version

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