Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#49 Feat/query all ever gifted punks #51

Merged
merged 10 commits into from
May 1, 2021

Conversation

eenagy
Copy link
Contributor

@eenagy eenagy commented Apr 30, 2021

This tab will show all the punks that have ever been gifted. Same rules of sorting by newest at the top, apply.
Timestamp needed to be added after merge #48
Also, created a new way to show crypto punks, which has a little overlapping on mobile. Let me know if the design is something preferable, and I will fix the mobile issue with it.

@vercel
Copy link

vercel bot commented Apr 30, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/renft/cryptopunks/C39Eg8PsYNfjCWDQteXBUtbZMPFd
✅ Preview: https://cryptopunks-git-fork-eenagy-feat-query-all-ever-gifte-5d946f.vercel.app

@nazariyv
Copy link
Member

nazariyv commented Apr 30, 2021

I LOVE the cleaner design. Didn't notice any mobile overlap :)

But I have a few very minor comments.

(1) The tab currently gifted isn't showing anything, but there are a number of punks being gifted right now. The definition of the punks that are being gifted right now is when the end tenancy date is in the future.

(2) The tab of all ever gifted isn't showing all the historical and current punks. (This could be a problem with the subgraph, whereby it deletes the punks that are no longer active)

@eenagy
Copy link
Contributor Author

eenagy commented Apr 30, 2021

I LOVE the cleaner design. Didn't notice any mobile overlap :)

The mobile overlap happens with longer hour values: 11:23 pm, it looks minor, but still.

But I have a few very minor comments.

(1) The tab currently gifted isn't showing anything, but there are a number of punks being gifted right now. The definition of the punks that are being gifted right now is when the end tenancy date is in the future.

done

(2) The tab of all ever gifted isn't showing all the historical and current punks. (This could be a problem with the subgraph, whereby it deletes the punks that are no longer active)

Maybe because the tenantId for the expired ones set back to "0x0000000000000000000000000000000000000000", but just guessing. I only get 6 punks back from the API alltogether.

@nazariyv
Copy link
Member

Maybe because the tenantId for the expired ones set back to "0x0000000000000000000000000000000000000000", but just guessing. I only get 6 punks back from the API altogether.

where have you noticed this?

@eenagy
Copy link
Contributor Author

eenagy commented Apr 30, 2021

Maybe because the tenantId for the expired ones set back to "0x0000000000000000000000000000000000000000", but just guessing. I only get 6 punks back from the API altogether.

where have you noticed this?

Queries, I see this filter:

  where: { tenant_not: "0x0000000000000000000000000000000000000000" }

Not sure if the tenant is set back, just see the filter.

@nazariyv
Copy link
Member

just created a query for all ever gifted. You can use the following

https://thegraph.com/explorer/subgraph/re-nft/cryptopunks-rent?query=All_Ever_Gifted

@nazariyv
Copy link
Member

nazariyv commented Apr 30, 2021

and here is the query, in case subgraph does not load for you:

{
    provenances {
      id
      minSalePriceInWei
      cryptopunk {
        id
      }
      tenancyDates {
        id
        start
      }
    }
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants