-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
wallet-ext: display non-transferable nfts #4784
wallet-ext: display non-transferable nfts #4784
Conversation
💳 Wallet Extension has been built, you can download the packaged extension here: https://github.com/MystenLabs/sui/actions/runs/3135108304#artifacts |
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.
Looks good
95a1d13
to
f6e0c80
Compare
|
303b9c2
to
3813e87
Compare
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.
Thanks @pchrysochoidis, PR behavior is aligned with what we agreed last week.
* allow nfts with `has_public_transfer` being false * disable send button when nft is not transferable * display NO MEDIA placeholder for nfts that don't have known media field * show object id when nft doesn't have a known name field
* add nft-type-image
* max length to 10 with max 6 chars prefix (default case for object ids)
* new background and icon * show tooltip only when hovering above the icon * wide view display `No media` as file type
3813e87
to
48190f2
Compare
has_public_transfer
being falsefixes: #4676