mirror of
https://github.com/status-im/status-desktop.git
synced 2025-02-13 07:06:58 +00:00
fix(JoinCommunity): Merge issues solved and channel hidden logic fixed
- Removed function in merge while it's still needed. - Modified logic when channels hidden to take into account also `viewOnly` permission cases.
This commit is contained in:
parent
4483eded3c
commit
af88cf2ef9
@ -58,7 +58,7 @@ Control {
|
|||||||
filters: [
|
filters: [
|
||||||
// The only permissions to be discarded are if they are private and NOT met
|
// The only permissions to be discarded are if they are private and NOT met
|
||||||
FastExpressionFilter {
|
FastExpressionFilter {
|
||||||
expression: { return !!model && !(!model.tokenCriteriaMet && model.isPrivate) }
|
expression: d.filterPermissions(model)
|
||||||
expectedRoles: ["tokenCriteriaMet", "isPrivate"]
|
expectedRoles: ["tokenCriteriaMet", "isPrivate"]
|
||||||
}
|
}
|
||||||
]
|
]
|
||||||
@ -90,6 +90,10 @@ Control {
|
|||||||
}
|
}
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function filterPermissions(model) {
|
||||||
|
return !!model && (model.tokenCriteriaMet || !model.isPrivate)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
padding: 35 // default by design
|
padding: 35 // default by design
|
||||||
@ -118,8 +122,18 @@ Control {
|
|||||||
model: d.visiblePermissionsModel
|
model: d.visiblePermissionsModel
|
||||||
}
|
}
|
||||||
|
|
||||||
|
StatusBaseText {
|
||||||
|
Layout.fillWidth: true
|
||||||
|
|
||||||
|
visible: root.allChannelsAreHiddenBecauseNotPermitted
|
||||||
|
horizontalAlignment: Text.AlignHCenter
|
||||||
|
wrapMode: Text.WordWrap
|
||||||
|
text: d.allChannelsAreHiddenBecauseNotPermittedText
|
||||||
|
textFormat: Text.StyledText
|
||||||
|
}
|
||||||
|
|
||||||
CustomHoldingsListPanel {
|
CustomHoldingsListPanel {
|
||||||
visible: !root.joinCommunity && d.viewOnlyPermissionsModel.count > 0
|
visible: (!root.joinCommunity && d.viewOnlyPermissionsModel.count > 0) && !root.allChannelsAreHiddenBecauseNotPermitted
|
||||||
introText: root.requiresRequest ?
|
introText: root.requiresRequest ?
|
||||||
qsTr("To view the <b>#%1</b> channel you need to join <b>%2</b> and prove that you hold").arg(root.channelName).arg(root.communityName) :
|
qsTr("To view the <b>#%1</b> channel you need to join <b>%2</b> and prove that you hold").arg(root.channelName).arg(root.communityName) :
|
||||||
qsTr("To view the <b>#%1</b> channel you need to hold").arg(root.channelName)
|
qsTr("To view the <b>#%1</b> channel you need to hold").arg(root.channelName)
|
||||||
@ -127,10 +141,9 @@ Control {
|
|||||||
}
|
}
|
||||||
|
|
||||||
CustomHoldingsListPanel {
|
CustomHoldingsListPanel {
|
||||||
visible: (!root.joinCommunity && d.viewAndPostPermissionsModel.count > 0) || root.allChannelsAreHiddenBecauseNotPermitted
|
visible: (!root.joinCommunity && d.viewAndPostPermissionsModel.count > 0) && !root.allChannelsAreHiddenBecauseNotPermitted
|
||||||
introText: root.allChannelsAreHiddenBecauseNotPermitted ? d.allChannelsAreHiddenBecauseNotPermittedText :
|
introText: root.requiresRequest ? qsTr("To view and post in the <b>#%1</b> channel you need to join <b>%2</b> and prove that you hold").arg(root.channelName).arg(root.communityName) :
|
||||||
root.requiresRequest ? qsTr("To view and post in the <b>#%1</b> channel you need to join <b>%2</b> and prove that you hold").arg(root.channelName).arg(root.communityName) :
|
qsTr("To view and post in the <b>#%1</b> channel you need to hold").arg(root.channelName)
|
||||||
qsTr("To view and post in the <b>#%1</b> channel you need to hold").arg(root.channelName)
|
|
||||||
model: d.viewAndPostPermissionsModel
|
model: d.viewAndPostPermissionsModel
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user