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

wip: MintRallyViewer.sol like ERC1155 without write functions #524

Open
wants to merge 3 commits into
base: staging
Choose a base branch
from

Conversation

yawn-c111
Copy link
Collaborator

編集中です!

MintRallyのNFT(MintNFT.sol)のview関数を使って、eventIdをtokenIdに見立ててERC1155かのように表示できるsolファイルを作成し、サンプルとして私個人の秘密鍵でMintRallyの全コントラクトとともにテストネットにデプロイしました!(stagingを汚さないために)

MintNFT サンプル:0x225B131690c2648EE58E7684e613C07D01A1B946
EventManager サンプル:0x71BAfD0812b483054b7e0c66dB428eB4AA54E13C
MintRallyViewer サンプル:0x283024996Bab0364A5Daa7a81A87eAF645a70eD6

私のdevアドレスのOpenseaページ:
https://testnets.opensea.io/ja/0xc5952da2d393d3421D56bd5FBCac1F8a3df40567

MintNFT サンプルのOpenseaページ:
https://testnets.opensea.io/ja/collection/mintrally-15

現状では、ERC1155のViewerをかませても、同じNFTがERC721とERC1155で2重に表示される現象は起きていません。(少し様子を見た方がいいかも?)

※ 私の手元では hardhat.config.ts の solidity version を "0.8.20" に設定してありますが、こちら影響が大きい気がしたので、プッシュはしてません。

Copy link

vercel bot commented Feb 28, 2024

@yawn-c111 is attempting to deploy a commit to the Code for Japan Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Feb 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mint-rally ❌ Failed (Inspect) Feb 28, 2024 11:59am

@yu23ki14
Copy link
Member

いいですね!
EventIDではなく、GroupIDでERC1155のようにみえるということをですかね?

@yawn-c111
Copy link
Collaborator Author

@yu23ki14
ありがとうございます!

現状はeventIdにしてあります!
MintNFTのisHoldingEventNFTを参照させてます
https://github.com/hackdays-io/mint-rally/blob/staging/hardhat%2Fcontracts%2FMintNFT.sol#L270-L276

ただ、書いてる途中でgroupIdにするのもありだなと感じてまして、用途によっては変更または併用(片方の関数名は独自命名)もいいかも…と思ってました!

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.

2 participants