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 pr) implement basic erc721 structure for certificates #14

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

Conversation

aradix16
Copy link

Gm! I'm sharing my current progress on the ERC721 certificate feature to get feedback on the structure and implementation.

  • Implemented a minimal mint function in components/certificate/certificate.cairo
  • Created contracts/certificate.cairo to initialize an ERC721 contract, aiding in testing
  • Added tests/tests_certificate.cairo file to verify token initialization and minting functionality.

This is a challenging task for me, but if you judge im going in the right direction, I think I'll be able to finish for the ODHack.
Thanks

@aradix16 aradix16 force-pushed the feat/nft-certificate branch from 589812d to 1650541 Compare December 3, 2024 07:13
@julienbrs julienbrs requested review from julienbrs and removed request for julienbrs December 3, 2024 09:39
@aradix16 aradix16 force-pushed the feat/nft-certificate branch from 22b6702 to 95a8fe3 Compare December 11, 2024 19:51
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.

1 participant