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

task3: veithly #787

Closed
wants to merge 4 commits into from
Closed

task3: veithly #787

wants to merge 4 commits into from

Conversation

veithly
Copy link
Contributor

@veithly veithly commented Jun 20, 2024

…rface

Add ERC20 and ERC721 smart contracts named MyERC20Token and MyNFT, respectively. Introduce the NFTMarket contract that facilitates buying and selling of NFTs, leveraging OpenZeppelin libraries for security and efficiency. README updated with deployed contract addresses for list and buy functionalities.

PR规范

注册的 PR,命名为 init: user-name,例如 init: Beavnvvv

提交 task 的 PR,命名为 task[n]: user-name,例如: task0: Beavnvvv

需要编写网页的 task,请在描述中上传效果截图。

请不要把报名和提交TASK的内容合并到一起,也不要将多个TASK放到一起提交!
请不要把报名和提交TASK的内容合并到一起,也不要将多个TASK放到一起提交!
请不要把报名和提交TASK的内容合并到一起,也不要将多个TASK放到一起提交!

…rface

Add ERC20 and ERC721 smart contracts named `MyERC20Token` and `MyNFT`, respectively.
Introduce the `NFTMarket` contract that facilitates buying and selling of NFTs,
leveraging OpenZeppelin libraries for security and efficiency. README updated with
deployed contract addresses for list and buy functionalities.
Copy link
Collaborator

@linghuccc linghuccc left a comment

Choose a reason for hiding this comment

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

买家,卖家请用不同地址

veithly added 3 commits July 4, 2024 18:01
Replace the old NFT contract addresses with the new ones for listing and buying
in the README.md file to reflect the most recent deployments.
Implement a new function `getAllListings` in the NFTMarket contract to allow
外部调用者获取所有上市的信息。此外, 更新了README以反映新的合约地址。
@veithly veithly closed this Jul 4, 2024
@linghuccc linghuccc self-assigned this Jul 5, 2024
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