maxrioux100 / gamestop-nft-helper Goto Github PK

History helper chart always shows account id even when username is available.

Steps to reproduce:

1. Ensure extension is installed & updated
2. Load an NTF at https://nft.gamestop.com/token/
3. Ensure that the NFT has >1 number of transactions
4. Ensure that the History helper chart is loaded
5. Mouse over the History helper chart.
6. Observe the chart labels

Current behavior:
The account ids are minimized (does not show the full account id).

Expected behavior:
If a username exists, it should show up in the mouse over event label.

How to resolve:
Use the value for the href instead of using the innerText value for each element under HistoryItemWrapper.p.span[n].strong.a where n=0 is the buyer and n=1 is the seller. Split the href value to only obtain the username.

useless since we have the volume graph now

Existing functionality:
the "Offers helper" chart is surfaced when the "Editions" button is clicked.

Proposed functionality:
"Offers helper" header should be created upon page load. Text should exist under this header to prompt the user to open the "Editions" button in order to load the data. Once the editions button has been clicked and the slider tray has been closed, the text prompt should be replaced with the Offers helper chart.

Opening NFT in new tab does not load the extension data

Steps to reproduce:

1. Navigate to NFT profile
2. Right click on NFT -> Open in new tab
3. Switch to new tab

Observed behavior:
There is no extension data (charts, tables, etc) added to the page.

Expected behavior:
The extension data (charts, tables, etc) should be added to the page.

When clicking on the "Editions" button and the chart appears, however there is no data available in the chart even though data was available in the "Editions" slider tray.

This NFT currently has 2 editions available to purchase:
https://nft.gamestop.com/token/0xa327d6f2c31db791ce2e491fbf3ad6c226999ea8/0x7d22f63b425a13e46981779fd749376f5d8196e52868d947516c5b9fbfc6fafc

This NFT currently has 2 editions available to purchase:
https://nft.gamestop.com/token/0xd609f93080383f759e41791bcf07f0b65bac651c/0xf4fddc5571d786fc486c09a81f99c92cfcec9fc1253c44af763593d42259fb35

When user is authenticated (signed in) and viewing an NFT that they own, it would be useful to have a dot within the history helper chart which represents the price point that the authenticated user purchased the NFT.

Requirements:

• If a user's name shows up as a buyer in the history helper, then a dot of a specific color is placed (perhaps green).
• If a user's name shows up as a seller in the history helper, then a dot of a different specific color is placed (perhaps red).

Note: If a user buys and sells an NFT multiple times, then there will be multiple colored dots.

Out of scope:
We will not be pairing multiple purchases with multiple sales.

The dollar sign is supposed to precede the value.

https://nft.gamestop.com/token/0x31585bc66347e5d1e8edf5af2a7bb864e9c04055/0x36cc66ec3315492400e5cd47049c9a2434237216e404a59cb00802b1082edcc1

and check the start of createOffersHelperContainer()

• Make offers helper an area chart
• Remove dataLabels (Note: labels will still show upon mouse hover event)

https://nft.gamestop.com/token/0x1d006a27bd82e10f9194d30158d91201e9930420/0x9de1bfa254668fdb9de296941782a7f95fc3e8a3fca9c13a08ac3f1c9155da96

The X axis labels are unnecessary in the Offers helper chart since this number is visible upon the mouse hover event. Futhermore, the X axis labels are overlaid on top of each other in NFTs with a large number of transactions, rendering them unusable.

This is a lower priority issue.

I noticed the Google Chrome extension had a few error messages, it seems that these are related to the Apex chart implementation.

This is the URL that is referenced in the error message context:
https://nft.gamestop.com/token/0xe78b7d9abacffe9616a36a2a2f31e5cd47b0db08/0x6361ee9e22851912610b33ccafa44bce4eef8bacd8b98dfac4bdc8de5976d368