search

status-im / status-desktop

Status Desktop client made in Nim & QML
https://status.app
Mozilla Public License 2.0
298 stars 79 forks source link

[Community] Join community process condition review #14213

Open noeliaSD opened 7 months ago

noeliaSD commented 7 months ago

Description

Join process is using JoinPermissionsOverlayPanel and this is handling a lot of different cases depending on the permission type, depending if the channels are hidden.. and we've been introducing more conditions to this flow without taking a photo of the complete picture and we can miss some scenarios and also the view is becoming complex.

Here I propose to do a diagram of ALL cases and do the proper refactor of this view to handle them without having a really fragil implementation as it is now.

noeliaSD commented 4 months ago

Moved to next milestone due to team capacity and priorities.