NFT Image is not displaying on Metamask, even Opensea

[ArnaudBand]

Lesson 11:

I finish deploying my NFT but the problem is not display an image after importing it on Metamsk and even on Opensea. Any help how to fix it

[EngrPips]

Hello @ArnaudBand , i ran into an issue where it wasn't displaying on metamask myself and didn't spend too much time to fix it before i moved on but of recently someone else ran into the same problem and they did figure it out and fix it and also they wrote an article on how they fix it on their linkedin profile, maybe that would help you troubleshoot this problem. Here https://www.linkedin.com/pulse/making-your-nft-collection-yevhen-hrytsenko-mjxif/ is the link if you would want to check out what they wrote and how they resolved the issue

[ArnaudBand]

Hi @EngrPips, I have an issue at the moment, I changed the info of IPFS using the LinkedIn info, I can't the mint the nft and I can see the the NFT on Opensea.

i got this error when 'm minting

  [6879] MintDogeNFT::run()
    ├─ [3925] DevOpsTools::get_most_recent_deployment("DOGIE", 11155111 [1.115e7]) [delegatecall]
    │   ├─ [0] VM::readDir("./broadcast", 3) [staticcall]
    │   │   └─ ← [Revert] the path broadcast is not allowed to be accessed for read operations
    │   └─ ← [Revert] the path broadcast is not allowed to be accessed for read operations
    └─ ← [Revert] the path broadcast is not allowed to be accessed for read operations


Error: 
script failed: the path broadcast is not allowed to be accessed for read operations
make: *** [Makefile:74: mint] Error 1

[EngrPips]

Hello @ArnaudBand, please add the line of code below to your foundry.toml file, and you should stop getting that error.

fs_permissions = [{ access = "read", path = "./broadcast" }, {access = "read", path = "./Images/"}]

[ArnaudBand]

Thank you @EngrPips, now the problem, is displaying that I didn't deploy the nft smart contract

  [451804] MintDogeNFT::run()
    ├─ [448856] DevOpsTools::get_most_recent_deployment("DOGIE", 11155111 [1.115e7]) [delegatecall]
    │   ├─ [0] VM::readDir("./broadcast", 3) [staticcall]
    │   │   └─ ← [Return] [("", "/workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol", 1, true, false), ("", "/workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol/11155111", 2, true, false), ("", "/workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol/11155111/run-1713432503.json", 3, false, false), ("", "/workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol/11155111/run-latest.json", 3, false, false)]
    │   ├─ [0] VM::toString(11155111 [1.115e7]) [staticcall]
    │   │   └─ ← [Return] "11155111"
    │   ├─ [0] VM::toString(11155111 [1.115e7]) [staticcall]
    │   │   └─ ← [Return] "11155111"
    │   ├─ [0] VM::toString(11155111 [1.115e7]) [staticcall]
    │   │   └─ ← [Return] "11155111"
    │   ├─ [0] VM::toString(11155111 [1.115e7]) [staticcall]
    │   │   └─ ← [Return] "11155111"
    │   ├─ [0] VM::readFile("/workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol/11155111/run-latest.json") [staticcall]
    │   │   └─ ← [Return] <file>
    │   ├─ [0] VM::parseJsonUint("<stringified JSON>", ".timestamp") [staticcall]
    │   │   └─ ← [Return] 1713432503 [1.713e9]
    │   ├─ [0] console::log("Processing: ", "/workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol/11155111/run-latest.json") [staticcall]
    │   │   └─ ← [Stop] 
    │   ├─ [0] VM::parseJson("<JSON file>", "$.transactions") [staticcall]
    │   │   └─ ← [Return] <encoded JSON value>
    │   ├─ [0] VM::keyExists("<JSON file>", "$.transactions[0].rpc") [staticcall]
    │   │   └─ ← [Return] false
    │   ├─ [0] console::log("Inspecting %s transactions", 1) [staticcall]
    │   │   └─ ← [Stop] 
    │   └─ ← [Revert] revert: No contract deployed
    └─ ← [Revert] revert: No contract deployed



== Logs ==
  Processing:  /workspaces/cyfrin_updraft_course/foundry_mood_nft/broadcast/DeployDogeNFT.s.sol/11155111/run-latest.json
  Inspecting 1 transactions
Error: 
script failed: revert: No contract deployed

[EngrPips]

Interesting, now show the NFT contract itself

[ArnaudBand]

// SPDX-Licenser-Identifier: MIT
pragma solidity ^0.8.19;

import "@openzeppelin/contracts/token/ERC721/ERC721.sol";

contract DogeNFT is ERC721 {
    error DogeNFT__TokenUriNotFound();

    mapping(uint256 tokenId => string tokenUri) private s_tokenURIs;
    uint256 private s_tokenCounter;

    constructor() ERC721("DOGIE", "DOG") {
        s_tokenCounter = 0;
    }

    function mintNFT(string memory tokenUri) public {
        s_tokenURIs[s_tokenCounter] = tokenUri;
        _safeMint(msg.sender, s_tokenCounter);
        s_tokenCounter = s_tokenCounter + 1;
    }

    function tokenURI(uint256 tokenId) public view override returns (string memory) {
        if (ownerOf(tokenId) == address(0)) {
            revert DogeNFT__TokenUriNotFound();
        }
        return s_tokenURIs[tokenId];
    }

    function getTokenCounter() public view returns (uint256) {
        return s_tokenCounter;
    }
}

[ArnaudBand]

Can I call you on discord?

[EngrPips]

Now the problem is glaring: the name of your contract in the above picture is DogeNFT, but when you used the DevOps tool in your mint script, you passed in DOGIE as shown in the line of code below.

address mostRecentlyDeployedDogieNft = DevOpsTools.get_most_recent_deployment("DOGIE", block.chainid);

So, change the line in your mint script to the below, and your code should now work fine.

address mostRecentlyDeployedDogieNft = DevOpsTools.get_most_recent_deployment("DogeNFT", block.chainid);

[ArnaudBand]

Thank you so much @EngrPips, I fix it

[EngrPips]

Awesome, I am glad you were able to figure it out. Keep crushing it, friend

[kinjun751]

I have the same issue the image doesnt show on metamask

[EngrPips]

I have the same issue the image doesnt show on metamask

Read this discussion #2112