Lesson-14 RandomNft compilation Error

[TimeKeyRoll]

This Error pops up at the time of compile. anyone know something on this?

Repo- https://github.com/TimeKeyRoll/hardhat-nft1-fcc/blob/master/contracts/RandomIpfsNft.sol

[othaime-en]

Hey @TimeKeyRoll there is an error in your imports on line 26.
Replace this line import "@openzeppelin/contracts/token/ERC721/extensions/ERC721URIStorage.sol";
with this line import "@openzeppelin/contracts/token/ERC721/ERC721.sol";

Also on line 34, change ERC721URIStorage to ERC721

[TimeKeyRoll]

I understand what you are saying But Patrick also used the same line because Of Extensions. I also have checked his repo . is there a change or update am I missing?

[othaime-en]

Oh yeah he used it. I don't see any problem with _safeMint at the moment. I need to look deeper into the contract.
In the meantime, line 80 and 104 need to changed. The NatSpec convention (the /** */ way of commenting) doesn't just allow any words to be used with the @ symbol. So change the @read to @notice in both lines.

[TimeKeyRoll]

Got it! Netspec error is fixed now. Just left with 'safe mint' error.

[TimeKeyRoll]

@othaime-en hey it got fixed. Thanks
BTW safeMint i was using = lol

[krakxn]

From the other thread, it seems you made some changes; is the repo. updated with them (before I clone)?

[TimeKeyRoll]

Question: why there is an Error when i try to compile randomNft.

Answer : I have no idea but I have posted it here to get the fix. So it is a all new question with a new repo and a brand new contract. do you know answer to this @krakxn

[krakxn]

Okay, I will look into it.

[krakxn]

First error:

Fixed by commenting out as follows,

Second:

These can be fixed by not using the @read for functions.

Third:

Fixed by removing the = as so

Warning:

Fixed by providing one.

Fourth:

Fixed by changing to dogBreed

Fifth:

Fixed by changing scope to view as so

Sixth:

Typo, fixed by correcting the typo as so

---

Result

Yay! It compiled. You can fix the warnings if you want as outlined above.

Incorporating the fixes above to counter the specific errors elaborated above should fix your compilation error.

Do you want me to make a PR for the ease of incorporating these changes? If not, you can do it manually as well.

[TimeKeyRoll]

@krakxn I am truly impressed like you really took time to have those snipptes included. This is really helpful and help me to understand why and what went wrong plus giving me the ability to fix it manually. And gives memory instance to brain cells. It is working and I am thankful. Appreciated!
P.S PR's are cool and a quick fix but i aint looking for them for now.

[krakxn]

Good luck! :)