status-desktop/ui
Jonathan Rainville 5c75c265af fix(permissions): fix hang when all channel perm check return (#14259)
* fix(permissions): fix hang when all channel perm check return

Fixes #14234

The problem was that we updated **all** the models from **all** the channels of a community each time the channel requirement checks returned.

The fix is to first of all, make sure we don't call that check too often. It sometimes got called twice in a row by accident.

The other better fix is to check if anything actually changed before updating. This solves the issue almost entirely.  Since the permissions almost never change, the updates now take only a second.

* fix(permisisons): never run permission checks for privileged users

Also fixes #14234 but for admins, TMs and Owners.

Admins+ were still getting the hang, because the permission checks always returned something different than the models, because the models knew that admins have access to everything, but the permission check was running as if it were a normal user (I think, un-tested).

Anyway, the solution is more simple, we never need  to run the permission checks on admins+, because they always have access to everything!

* fix(Communities): prevent channels model from emitting unnecessary signals

Closes: #14274

* chore(Communities): improve channels metadata lookup performance

ChannelsSelectionModel is removed, replaced with plain LeftJoinModel.
Transformations of left-side model are done in a single place, not in
every delegate making the join.

* only call update functions when there is something to update + move permission model creation when needed

---------

Co-authored-by: Michał Cieślak <michalcieslak@status.im>
2024-04-04 12:14:39 -04:00
..
StatusQ chore(SharedAddressesPermissionsPanel): optimize the speed 2024-04-04 16:59:26 +02:00
app fix(permissions): fix hang when all channel perm check return (#14259) 2024-04-04 12:14:39 -04:00
fonts fix(cpp): make libraries which are not dynamically loaded static 2022-08-02 15:30:08 +02:00
i18n chore: update base and English plural translations 2024-03-18 09:43:06 +01:00
imports fix(@desktop/communities): request to join fixes (#14287) 2024-04-04 18:03:10 +02:00
include/StatusDesktop/Monitoring chore(general): move cpp files of monitoring tool from DOtherSide to ui dir 2023-02-09 14:27:53 +01:00
src/StatusDesktop/Monitoring chore(general): move cpp files of monitoring tool from DOtherSide to ui dir 2023-02-09 14:27:53 +01:00
MonitoringSources.cmake chore(general): move cpp files of monitoring tool from DOtherSide to ui dir 2023-02-09 14:27:53 +01:00
generate-rcc.go chore(@desktop/walletconnect): communication between qt desktop app and loaded wallet connect sdk is made via web channel 2023-11-17 13:55:20 +01:00
main.qml fix: 150% and 200% zoom levels prevent the user from logging in 2024-02-27 10:39:05 +01:00
nim-status-client.pro feat(general): Initial setup for app monitoring tool 2023-01-13 13:47:08 +01:00