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
fix(TokenUI): Hide explicitly ManageTokensCommunityTag in community settings page #14500
fix(TokenUI): Hide explicitly ManageTokensCommunityTag in community settings page #14500
Conversation
Jenkins BuildsClick to see older builds (18)
|
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.
I've tested the flow and it works properly but I have some comments about the solution!
ui/app/AppLayouts/Wallet/views/collectibles/CollectibleView.qml
Outdated
Show resolved
Hide resolved
Checking Figma Communities P3 -> Tokens , it looks like indeed communityId should not be set on MintedTokensView component. |
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.
Tested again and works fine!
Thanks for addressing the comments! I've added some other minor ones, please take a look. Otherwise, LGTM!
You will need to rebase on top of master to make the e2e tests work properly since there have been some changes there that are making this pr failing in this job!
@@ -178,6 +178,7 @@ Control { | |||
} | |||
|
|||
ManageTokensCommunityTag { | |||
id: manageTokenCommunityTag |
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.
Do we need then this id here?
1570e50
to
a3cb7c0
Compare
…ettings page according to collectibles cells sizing
a3cb7c0
to
6efc043
Compare
What does the PR do
PR hide the ManageTokensCommunityTag item in "Community"->Tokens page. Previous changes introduced ManageTokensCommunityTag 12519 ManageTokensCommunityTag for displaying community icon and name when showing Collectibles in User's Wallet->Collectibles page. However when going to particular Community and checking its "Tokens" there is no need to display that tag, according to the MintedTokensSettingsPanel PR: 11347
Affected areas
Wallet
Communities
StatusQ checklist
Screenshot of functionality (including design for comparison)
In Wallet -> Collectibles, ManageTokensCommunityTag is displayed: