-
Notifications
You must be signed in to change notification settings - Fork 76
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
13291 hide channels #13876
13291 hide channels #13876
Conversation
Jenkins BuildsClick to see older builds (12)
|
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.
Good job
@@ -227,6 +263,18 @@ QtObject: | |||
defer: index.delete | |||
self.dataChanged(index, index, @[ModelRole.CategoryOpened.int]) | |||
|
|||
proc changeCategoryItemIfHiddenBecauseNoPermitted*(self: Model, id: string) = |
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 guess you made a typo?
proc changeCategoryItemIfHiddenBecauseNoPermitted*(self: Model, id: string) = | |
proc changeCategoryItemIsHiddenBecauseNotPermitted*(self: Model, id: string) = |
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!
So much bootstraping needed for a single new property 😢
eca5b7b
to
eca9b29
Compare
…d to view and hideIfPermissionsNotMet is set Extend chats model with channel permissions info and hideIfPermissionsNotMet. Visibility of chat item is based on: member roles, channel permissions, hideIfPermissionsNotMet. If all channels from category are hidden, category item is also hidden. If all chats in community are hidden, infomration label is displayed. Issue #13291
eca9b29
to
0556f69
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.
Nice work
* feat(@desktop/communities): Hide channels if the user is not permitted to view and hideIfPermissionsNotMet is set Extend chats model with channel permissions info and hideIfPermissionsNotMet. Visibility of chat item is based on: member roles, channel permissions, hideIfPermissionsNotMet. If all channels from category are hidden, category item is also hidden. If all chats in community are hidden, infomration label is displayed. Issue #13291 * chore(@desktop): Upgrade status-go Issue #13291
What does the PR do
Extend chats model with channel permissions info and hideIfPermissionsNotMet.
Visibility of chat item is based on: member roles, channel permissions, hideIfPermissionsNotMet.
If all channels from category are hidden, category item is also hidden.
If all channels in community are hidden, information label is displayed.
Related status-go pr
Issue #13291
hide-channels.mov