Skip to content

Wisdom#14

Open
enyinnaya1234 wants to merge 2 commits intoBlockheaderWeb3-Community:mainfrom
enyinnaya1234:wisdom
Open

Wisdom#14
enyinnaya1234 wants to merge 2 commits intoBlockheaderWeb3-Community:mainfrom
enyinnaya1234:wisdom

Conversation

@enyinnaya1234
Copy link

@enyinnaya1234 enyinnaya1234 commented Feb 19, 2025

This is my implementation of a crowdfunding contract written in solidity: Link here [https://github.com/BlockheaderWeb3-Community/cohort-6/blob/main/assignments/Assignment3.md]

PR Checklist

  • Tests
  • Documentation

Copy link
Collaborator

@ibsule ibsule left a comment

Choose a reason for hiding this comment

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

Update PR title to contain a one-line summary of your PR

Copy link
Collaborator

@ibsule ibsule left a comment

Choose a reason for hiding this comment

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

Please deploy contracts and providing sample transaction as stated in 4.
image

pragma solidity ^0.8.24;

interface IERC721{
function _mint(address to, uint256 tokenId) external;
Copy link
Collaborator

Choose a reason for hiding this comment

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

By convention, external functions are not named with an underscore.
Update _mint() to mint()

Copy link
Author

Choose a reason for hiding this comment

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

yes sir...

God bless the instructor sir

@enyinnaya1234
Copy link
Author

Please deploy contracts and providing sample transaction as stated in 4. image

pls, should i send it as a screenshot?

@ibsule
Copy link
Collaborator

ibsule commented Feb 24, 2025

Send the transaction hash

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