Replies: 12 comments
-
@J-Son89, just to clarify, do I correctly understand that we show "Community owner node is offline" only in case when the community is free to join (no approve required) and our request is till pending)? |
Beta Was this translation helpful? Give feedback.
-
@VladimrLitvinenko I think it's good to get this clarification in public and also helps alongside the context of the Github issue, but can you raise this question in Mobile-ui so that the design team can see it too. They don't really use Github. From what I understand we just want to show this message to a user at the same time their request is pending. So this can be in all 4 cases. i.e this can be in both token-gated and non-token gated communities and when manual approval is needed and not needed. |
Beta Was this translation helpful? Give feedback.
-
@J-Son89, I believe that in cases when a user request should be accepted by a community owner, this message would be both incorrect and redundant. Incorrect - because maybe community owner IS online, but simply don't want you in the community) Redundant - because there is "Pending" badge. Wdyt? |
Beta Was this translation helpful? Give feedback.
-
let's have this conversation on Status - mobile ui channel so design team is brought into the conversation :) |
Beta Was this translation helpful? Give feedback.
-
Asked there) |
Beta Was this translation helpful? Give feedback.
-
this can be closed as the info box was added in #14515 |
Beta Was this translation helpful? Give feedback.
-
Not yet, unfortunately) In the Figma designs, info box shows up only for communities that are free to join and only in cases when the community owner node is offline. So current implementation shows infobox when: the community is open to join + user issued a join request + user didn't yet join the community. But here is the problem.. in our code community gets |
Beta Was this translation helpful? Give feedback.
-
The |
Beta Was this translation helpful? Give feedback.
-
This issue relates to communities that do not require admin approval |
Beta Was this translation helpful? Give feedback.
-
Both of them requires approval looking from the comments on the code https://github.com/status-im/status-go/blob/dd8be1d8b041f967a0707cfb70ac96c5a71d93a8/protocol/communities/community.go#L1205 , it might be that we are not sending a request to join in theory, have you checked? @0x-r4bbit might also been able to help you |
Beta Was this translation helpful? Give feedback.
-
Ah, I see! Let me dig deeper) |
Beta Was this translation helpful? Give feedback.
-
@vkjr, do you know if this discussion is still relevant, or better yet, is there a problem we need to solve or has it been solved since December last year? There are some questions you asked that I personally don't know the answer, so maybe they're still relevant and we could ask on the |
Beta Was this translation helpful? Give feedback.
-
We can't actually check when the owner is online, instead we will only show this "owner not online" box whenever there is a pending request. With the intention of optimising this approach at a later point.
Add notice to user that the user node is offline as per the designs.
https://www.figma.com/file/h9wo4GipgZURbqqr1vShFN/Communities-for-Mobile?node-id=7035%3A462899&t=ivg9IboexBv4dlSR-0
add integration tests to this.
Beta Was this translation helpful? Give feedback.
All reactions