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

Change the Join common\space flow #1940

Closed
3 of 5 tasks
elatif2020 opened this issue Aug 8, 2023 · 11 comments · Fixed by #1964 or #2068
Closed
3 of 5 tasks

Change the Join common\space flow #1940

elatif2020 opened this issue Aug 8, 2023 · 11 comments · Fixed by #1964 or #2068
Labels
Priority - A V [production] verified on production enviroment

Comments

@elatif2020
Copy link
Collaborator

elatif2020 commented Aug 8, 2023

Currently if a user try to open a link for a common\space\stream she isn't part of she is redirected to the backpage of the relevant page, from which she can join.
We want to change it, in the case that
(1) For a space\ stream in a space: The admittance to the space is automatic (as defined in the relevant assign circle proposal's parameters), and the user is a member of the relevant root-common.
(2) For a (root) common\ stream in a common: The admittance to the common is automatic.

We will show the relevant feed item if it has public visibility setting, with join button instead of the inputBox as in the design here

If the relevant item is a space\common we should show similarly the Home feed item for this space\common.

  • Add chat module design for the public view (user can see the chat, and with join button instead of the input box)

  • Enable public view of the feed (show only items with public visibility)

  • Disable redirection to the back page (if there is at least one public item) (if it is a space, only for parent-common members)

  • If space admittance is automatic : skip the "join space" popup (only for space, not common) #1951

  • TBD ? Change default proposal visibility according to the default of the common ?

@roienatan roienatan self-assigned this Aug 10, 2023
@roienatan roienatan linked a pull request Aug 14, 2023 that will close this issue
3 tasks
@roienatan
Copy link
Collaborator

@elatif2020 regarding the 3rd point: "Disable redirection to the back page (if there is at least one public item) (if it is a space, only for parent-common members)." We always have the Home card which is public. I think we should completely disable the auto redirection. Let me know what you think please.

@elatif2020
Copy link
Collaborator Author

@elatif2020 regarding the 3rd point: "Disable redirection to the back page (if there is at least one public item) (if it is a space, only for parent-common members)." We always have the Home card which is public. I think we should completely disable the auto redirection. Let me know what you think please.

not exactly, the Home could be be not public

@roienatan
Copy link
Collaborator

For the current progress and what is left to be done see my comment in the PR.

@andreymikhadyuk
Copy link
Collaborator

@elatif2020 I just realised that we should have info whether the common has public feed items or not in the common model or somewhere else. Because we cannot check all the feed items to decide whether redirect user or not.

@andreymikhadyuk
Copy link
Collaborator

@elatif2020 we cannot allow guests to see feed at the moment, because BE does not allow to fetch the feed items and pinned ones when user is not authenticated.

@elatif2020
Copy link
Collaborator Author

@elatif2020 I just realised that we should have info whether the common has public feed items or not in the common model or somewhere else. Because we cannot check all the feed items to decide whether redirect user or not.

Created a BE issue for this
https://github.com/daostack/common-backend/issues/2049

@elatif2020
Copy link
Collaborator Author

elatif2020 commented Sep 11, 2023

  • Change the Join button text to just "Join" (both for a space and for a common)
  • Disable showing the front page for a space for which the user isn't a member of the parent common.
    Show the space backpage in this case.

@budnik9 budnik9 self-assigned this Sep 11, 2023
@elatif2020 elatif2020 reopened this Sep 11, 2023
@budnik9
Copy link
Contributor

budnik9 commented Sep 11, 2023

@elatif2020 What is front page and backpage for a space? Could you please provide me examples?
image

@elatif2020
Copy link
Collaborator Author

@elatif2020 What is front page and backpage for a space? Could you please provide me examples? image

@budnik9
Sure, sorry it wasn't clear.

This is the frontpage:
image

And this is the backpage:
image

@budnik9
Copy link
Contributor

budnik9 commented Sep 12, 2023

@elatif2020 Should we check if the user is a member only of the direct parent common or all parent commons to show backpage / frontpage for a user?
For example, we have this structure:

L1 common
    L2 space
        L3 space
            L4 space

For L4 space should we check only L3 space or all ancestors (L3, L2, L1)? Because now I can leave (for example) L2 space without leaving L3 and L4 spaces. Maybe this is a bug.

@andreymikhadyuk andreymikhadyuk removed their assignment Sep 12, 2023
@elatif2020
Copy link
Collaborator Author

elatif2020 commented Sep 12, 2023

@elatif2020 Should we check if the user is a member only of the direct parent common or all parent commons to show backpage / frontpage for a user?
For example, we have this structure:

L1 common
    L2 space
        L3 space
            L4 space

For L4 space should we check only L3 space or all ancestors (L3, L2, L1)? Because now I can leave (for example) L2 space without leaving L3 and L4 spaces. Maybe this is a bug.

@budnik9
only direct parent

@budnik9 budnik9 linked a pull request Sep 12, 2023 that will close this issue
4 tasks
andreymikhadyuk added a commit that referenced this issue Sep 13, 2023
…mmon-flow

Change the Join common\space flow #1940
@NoamQA NoamQA added the V [production] verified on production enviroment label Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority - A V [production] verified on production enviroment
Projects
None yet
5 participants