-
-
Notifications
You must be signed in to change notification settings - Fork 353
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
#3203 add table to show currently standing offers for user #3477
Conversation
✅ Deploy Preview for koda-nuxt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
I would sort the newest offers on top by default. Can we make columns sortable for amount, date and name? |
Can we please make the first column shortened? Perhaps add ... if the name of the nft is too long? Table seems to not be centered. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left the comment about first column, I guess I meant it in the similar fashion to https://beta.kodadot.xyz/bsx/stats
done |
@yangwao done |
wfm, just one thing, did we leave out the sorting by offer status on purpose? Maybe it'd be nice to have option to show only active offers since those are the ones which will be most interesting to user? @yangwao wdyt |
seems works on https://beta.kodadot.xyz/bsx/stats/ so let's have it there @prachi00
legit, add it here |
Added sorting in status @yangwao @petersopko |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wfm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yangwao I see that you've added this reuqest
did you mean to add some column which will track whether NFT is burned or not? Cause the query right now, is fetching all of the NFTs regardless of their state (so you can see the burned ones in the table) |
Let's make a separate issue
✅ |
yeah I can make that but kinda need more clarity on this one |
pay 250 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type
What's new?
Before submitting Pull Request, please make sure:
Optional
Had issue bounty label?
Payout
Community participation
Screenshot