Open justinasfour04 opened 3 years ago
@ethlocker are you the correct person to look at that issue ?
@ethlocker @sophoah have fixed this issue in this PR, please review and merge - https://github.com/harmony-one/chrome-extension-wallet/pull/109
When should we see the changes in the extension? Do we have to manually update?
@justinasfour04 have raised this with on Twitter - https://twitter.com/DatPenguinClub/status/1454099046944149505?s=20
Retweet and remind them to update the release
@datpenguinclub is it updated ?
@justinasfour04 yes mate - it's updated - also our pre-sale is live at https://datpenguinclub.com . Come and mint yourself a π§ and view them in the browser extension wallet in all their glory. ππ
also @justinasfour04 chrome automatically updates all extensions, so I don't think you have to manually update it - but the version that has the fix is 1.2.5
- the latest one
It doesn't seem to be working. I still get the NFT icon
@justinasfour04 could you right click your chrome extension and press "Manage Extension" and make sure that your wallet is on the latest version - 1.2.5
@justinasfour04 cool - but with the latest version - with my fix - I'm able to view the NFT along with their attributes
I am not
I guess I'll fork it and make the change myself and then add to review
@justinasfour04 - could you exit out of the pop-up and open it back up again if it renders - if not it maybe that the asset associated with your NFT is no longer available
@justinasfour04 Well there's no change to be made - this was the change that was preventing the render https://github.com/harmony-one/chrome-extension-wallet/pull/109
It works perfectly fine on Metamask, why does this have to be any different. Also I know that all the metadata is there for those NFTs I have
stuck forever loading
that's on the right path
close the pop-up and open it back up again
I am doing that
@justinasfour04 - I reckon the only solution left is uninstalling the chrome extension and re-installing it
@justinasfour04 and to check if it works - come mint a π§ at https://datpenguinclub.com π
I'm not minting a penguin just to see that it works.
Removed extension and added it back. Not a single NFT loads
@justinasfour04 well you could try it on the testnet - https://test.datpenguinclub.com - use the faucet https://faucet.pops.one/ to give yourself some ONE on the test network - and you could mint a penguin on test.datpenguinclub.com on the harmony testnet
Why are you shilling penguin. I'm telling it's not a good fix
@justinasfour04 I'm not shilling you my penguin - it costs you $0 to test it on the test network.
How about you test it with other nfts like pixel pirates or the frey or harmonauts
@justinasfour04 if you don't think it's a proper fix - you're welcome to give a better fix. I was just trying to tell you that my fix seems to fix not being able to view NFTs (attached screenshot above)..
Are any of these other projects like pixel pirates / freys and the harmonauts on the harmony testnet ?? if yes what's their URL ?? More than happy to test them out
@justinasfour04 I'm just trying to tell you that this issue has been solved - and I have screenshots both on the testnet and mainnet - I'm not trying to shill you anything - not an opinion - a fact
That's an explorer page - I can't mint anything on there - I need URLs to the projects test environments running on the harmony testnet - not the explorer page.
@justinasfour04 Do you have the URL to testnet version of harmonauts / or freys / or any other Dapps you mentioned ?? I mentioned mine - cause I have a test version of my Dapp running on the harmony testnet - so you can test it without any real $$.
Cause we'd like to have end to end integration and best UX when users do use our Dapp. So I'm not shilling you anything, in fact no one is asking me to any of this - I'm just here trying to help. I'm signing off for now πΆπ€·ββοΈ
Issue fixed in the following https://github.com/harmony-one/chrome-extension-wallet/pull/111
@datpenguinclub @givp @ethlocker @sophoah
I'm not sure if this is the issue but it seems like if the nft.uri is missing the following place holder gets displayed https://github.com/harmony-one/chrome-extension-wallet/blob/cc1e524f5df3caf789ccc2a48a48042079398944/src/popup/pages/Token/HRC721/Gallery.vue#L21
The nfts of Harmonauts are missing the URI but I looked at the 721 protocol and it doesn't seem like it's needed. Can someone look into why the NFT images are not appearing on the wallet
https://github.com/harmony-one/chrome-extension-wallet/blob/cc1e524f5df3caf789ccc2a48a48042079398944/src/services/Hrc721Service.js#L20 This function is getting called in the vue to get the uri given the owner id. Something is preventing it from retrieving the uri