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

feat(community): add option to see community rules #17022

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jrainville
Copy link
Member

What does the PR do

Fixes #16826

Adds the option to see the community rules in the community context menu on the left.

Also adapts a few things according to designs.

Affected areas

Community context menu

Architecture compliance

Screenshot of functionality (including design for comparison)

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

Design:
image

App:
image

Impact on end user

It's now possible to see the community rules after joining

How to test

  • Right click a community a click "View community rules"

Risk

Tick one:

  • Low risk: 2 devs MUST perform testing as specified above and attach their results as comments to this PR before merging.
  • High risk: QA team MUST perform additional testing in the specified affected areas before merging.

Worst case: 🤷

@jrainville jrainville requested review from micieslak, caybro, alexjba and a team as code owners January 3, 2025 19:27
@status-im-auto
Copy link
Member

status-im-auto commented Jan 3, 2025

Jenkins Builds

Click to see older builds (7)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 04a9a3b #1 2025-01-03 19:33:56 ~6 min macos/aarch64 🍎dmg
✔️ 04a9a3b #1 2025-01-03 19:34:24 ~6 min tests/nim 📄log
✔️ 04a9a3b #1 2025-01-03 19:39:38 ~12 min tests/ui 📄log
✔️ 04a9a3b #1 2025-01-03 19:43:23 ~15 min linux-nix/x86_64 📦tgz
✔️ 04a9a3b #1 2025-01-03 19:44:16 ~16 min macos/x86_64 🍎dmg
✔️ 04a9a3b #1 2025-01-03 19:45:36 ~18 min linux/x86_64 📦tgz
✔️ 04a9a3b #1 2025-01-03 19:52:56 ~25 min windows/x86_64 💿exe
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ a4c2fd2 #2 2025-01-06 15:48:43 ~4 min macos/aarch64 🍎dmg
✔️ a4c2fd2 #2 2025-01-06 15:51:54 ~7 min tests/nim 📄log
✔️ a4c2fd2 #2 2025-01-06 15:55:45 ~11 min tests/ui 📄log
✔️ a4c2fd2 #2 2025-01-06 15:57:15 ~13 min linux-nix/x86_64 📦tgz
✔️ a4c2fd2 #2 2025-01-06 15:59:24 ~15 min macos/x86_64 🍎dmg
✔️ a4c2fd2 #2 2025-01-06 16:00:20 ~16 min linux/x86_64 📦tgz
✔️ a4c2fd2 #2 2025-01-06 16:08:12 ~24 min windows/x86_64 💿exe
✔️ 46652fe #3 2025-01-08 18:23:40 ~5 min macos/aarch64 🍎dmg
✔️ 46652fe #3 2025-01-08 18:27:46 ~9 min tests/nim 📄log
✔️ 46652fe #3 2025-01-08 18:31:29 ~13 min tests/ui 📄log
✔️ 46652fe #3 2025-01-08 18:32:32 ~14 min linux-nix/x86_64 📦tgz
✔️ 46652fe #3 2025-01-08 18:34:29 ~16 min macos/x86_64 🍎dmg
✔️ 46652fe #3 2025-01-08 18:36:12 ~18 min linux/x86_64 📦tgz
✔️ 46652fe #3 2025-01-08 18:44:03 ~25 min windows/x86_64 💿exe

Copy link
Member

@caybro caybro left a comment

Choose a reason for hiding this comment

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

LGTM in general

ui/app/mainui/Popups.qml Outdated Show resolved Hide resolved
@jrainville jrainville force-pushed the feat/view-community-rules branch from 04a9a3b to a4c2fd2 Compare January 6, 2025 15:43
@jrainville
Copy link
Member Author

@caybro thanks for the review. Your points are fixed

ui/app/mainui/AppMain.qml Outdated Show resolved Hide resolved
Fixes #16826

Adds the option to see the community rules in the community context menu on the left.

Also adapts a few things according to designs
@jrainville jrainville force-pushed the feat/view-community-rules branch from a4c2fd2 to 46652fe Compare January 8, 2025 18:17
@jrainville jrainville requested review from a team and iurimatias and removed request for a team January 9, 2025 15:02
@jrainville
Copy link
Member Author

@alexjba sorry, you've been selected again 😅 It seems like the randomizer likes you a lot. To be fair, you test well, so it's good 😄

There were 4 items in your list. Here they are in random order:

alexjba
caybro
micieslak
iurimatias

@jrainville
Copy link
Member Author

@alexjba friendly reminder to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mobile parity: Add "View community rules" button
4 participants