-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add lists of NFTs in treasury detail page #2973
Comments
Hi @bonustrack, I'll work on this |
Hey @findtoni and @bonustrack I actually started working on this. Would you mind choosing any other task? |
Hey @findtoni, Thanks for letting me take this task. I actually contacted him on discord beforehand but forgot to mention it here. |
Yeah I just saw that, no worries. |
Hi @Bivab111 what's the status on this issue? Did you manage to draft a sketch of the UI update? |
@zzuziak sorry for late reply. I had the Design ready, but I could not complete a draft |
Thanks for a quick reply! What do you mean by not being able to complete a draft? Draft of the design or the draft PR? It would be great to see the design before you dig into the codebase 🙏 |
Hey @Bivab111 coming back on this one 😊 |
We should use OpenSea API to query the list of NFTs, we are in process of getting an API key we don't have one yet, but integration can be tested using testnet API (which doesnt require API key).
We will need a design for this, most likely it would use cards, similar than OpenSea but following our current style (without shadow for example)
The text was updated successfully, but these errors were encountered: