Skip to content
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: can't join an encrypted community #14518

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

caybro
Copy link
Member

@caybro caybro commented Apr 25, 2024

What does the PR do

  • fix the corner case and allow the user to join a community without an explicitly stated "Member" permission
  • enable/disable the Share/Join/Save buttons when the permission to join check is ongoing, or when the permission check failed
  • display tooltips over the disabled buttons explaining why it's disabled
  • always display the eligibility button floating on top of the (scrollable) contents

Fixes #14473
Fixes #14299

Affected areas

CommunityMembershipSetupDialog

Screenshot of functionality (including design for comparison)

  • I've checked the design and this PR matches it

Main join dialog:
Snímek obrazovky z 2024-04-25 12-24-03

Check over:
Snímek obrazovky z 2024-04-25 12-24-11

Editting addresses:
Snímek obrazovky z 2024-04-25 12-22-14

Snímek obrazovky z 2024-04-25 12-24-20

@caybro caybro linked an issue Apr 25, 2024 that may be closed by this pull request
@status-im-auto
Copy link
Member

status-im-auto commented Apr 25, 2024

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ b8037c2 #1 2024-04-25 10:00:46 ~6 min macos/aarch64 🍎dmg
✔️ b8037c2 #1 2024-04-25 10:04:59 ~10 min tests/nim 📄log
✔️ b8037c2 #1 2024-04-25 10:06:43 ~12 min macos/x86_64 🍎dmg
✔️ b8037c2 #1 2024-04-25 10:11:41 ~17 min tests/ui 📄log
✔️ b8037c2 #1 2024-04-25 10:19:11 ~24 min linux/x86_64 📦tgz
✔️ b8037c2 #1 2024-04-25 10:24:04 ~29 min windows/x86_64 💿exe
✔️ 1008058 #2 2024-04-25 15:22:36 ~8 min macos/aarch64 🍎dmg
✔️ 1008058 #2 2024-04-25 15:23:08 ~8 min tests/nim 📄log
✔️ 1008058 #2 2024-04-25 15:25:12 ~10 min macos/x86_64 🍎dmg
✔️ 1008058 #2 2024-04-25 15:31:20 ~16 min linux/x86_64 📦tgz
✔️ 1008058 #2 2024-04-25 15:31:54 ~17 min tests/ui 📄log
✔️ 1008058 #2 2024-04-25 15:41:36 ~26 min windows/x86_64 💿exe

@caybro caybro marked this pull request as ready for review April 25, 2024 10:37
- fix the corner case and allow the user to join a community without an
explicitely stated "Member" permission
- enable/disable the Share/Join/Save buttons when the permission to join
check is ongoing, or when the permission check failed
- display tooltips over the disabled buttons explaining why it's disabled
- always display the eligibility button floating on top of the
(scrollable) contents

Fixes #14473
Fixes #14299
@caybro caybro force-pushed the 14473-cant-join-an-encrypted-community branch from b8037c2 to 1008058 Compare April 25, 2024 15:14
Copy link
Contributor

@MishkaRogachev MishkaRogachev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

loading: root.requirementsCheckPending
type: d.lostCommunityPermission || d.lostChannelPermissions ? StatusBaseButton.Type.Danger : StatusBaseButton.Type.Normal
tooltip.text: {
if (interactive)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is tooltip available if !interactive

Copy link
Contributor

@alexjba alexjba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

ui/StatusQ/src/permissionutilsinternal.cpp Show resolved Hide resolved
@caybro caybro merged commit 0c8231a into master Apr 26, 2024
8 checks passed
@caybro caybro deleted the 14473-cant-join-an-encrypted-community branch April 26, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants