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

💄 UI updates for Primitive NFT #790

Merged
merged 14 commits into from
Nov 22, 2022
Merged

💄 UI updates for Primitive NFT #790

merged 14 commits into from
Nov 22, 2022

Conversation

nwingt
Copy link
Member

@nwingt nwingt commented Nov 19, 2022

image

image

src/locales/en.json Outdated Show resolved Hide resolved
src/locales/en.json Outdated Show resolved Hide resolved
src/locales/zh-Hant.json Outdated Show resolved Hide resolved
src/util/nft.js Outdated Show resolved Hide resolved
!this.sortedCollectedNFTs.length
!this.isCurrentTabCreated &&
// NOTE: Seems computed property `this.nftClassListOfCollected` is not reflecting the actual state
!this.nftClassListMap?.collected.length
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nftClassListOfCollected not working only in here?
Does it work if this handler is in nextTick?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tried, it doesn't work neither

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

strange, should spend time to investigate when free

src/mixins/nft.js Outdated Show resolved Hide resolved
nwingt and others added 2 commits November 22, 2022 01:08
Co-authored-by: William Chong <6198816+williamchong@users.noreply.github.com>
Co-authored-by: William Chong <6198816+williamchong@users.noreply.github.com>
src/util/nft.js Outdated Show resolved Hide resolved
@nwingt nwingt requested a review from williamchong November 21, 2022 17:12
@nwingt nwingt merged commit decbf2c into develop Nov 22, 2022
@nwingt nwingt deleted the feature/pnft branch November 22, 2022 03:15
Comment on lines -103 to +112
isWritingNFT() {
return isWritingNFT(this.NFTClassMetadata);
nftClassCollectionType() {
return getNFTClassCollectionType(this.NFTClassMetadata);
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems some conditional checks in if (this.isWritingNFT) should be refactored to if (this.nftClassCollectionType === nftClassCollectionType.WritingNFT)?
like here

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

Successfully merging this pull request may close these issues.

3 participants