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

Draft projects should not show up in the search bar for user who created it #581

Closed
darechan opened this issue Mar 7, 2023 · 11 comments · Fixed by #775
Closed

Draft projects should not show up in the search bar for user who created it #581

darechan opened this issue Mar 7, 2023 · 11 comments · Fixed by #775
Assignees
Labels
bug Something isn't working good first issue Good for newcomers low priority

Comments

@darechan
Copy link
Contributor

darechan commented Mar 7, 2023

The Bug

Projects created in draft mode by a user show up in search results for them on the landing page through the search bar

To Reproduce

Expected behavior

Draft projects should not show up in the search results

@darechan
Copy link
Contributor Author

darechan commented Mar 7, 2023

@tuxology @srish Hey! Please take a look at this issue. I'd love to be assigned to this issue if possible!

@khallow7
Copy link

khallow7 commented Mar 7, 2023

Yes, I found these issues as well. Plus, other things are not behaving the way they should. .

When I set the project to the “only me” option, the project is still visible to “authenticated users” → The project should only be seen by me, or it should move to drafts.

When I set the project to “only a few Zubhub users” it sends the project to draft, and it's only visible to me. → The project should be visible to authenticated users that you can select.

When I try to set “for all registered Zubhub users” then an additional field opens to input usernames. → The project should be visible to all the registered/authenticated users.

The labels names are not rightly ordered. These need to be fixed.

More details here:
#556 (comment)

@tuxology
Copy link
Member

tuxology commented Mar 7, 2023

@darechan Assigned to you. The correct behavior is what @khallow7 suggests. Note that the ZubHub hosted on https://zubhub.unstructured.studio is in Public mode. ON your local install you can check mode using Django Admin interface. Check the behavior of these project modes according to what @khallow7 suggests and verify if it works correctly with Public mode Zubhub as well. The issue most probably lies there, i.e. the behavior of various project viewing modes is not defined correctly in public mode Zubhub.

⚠️ Note: @darechan Please follow discussion on #556 closely. This issue not about UX, it is about functionality of viewing modes - specifically draft projects of other users ending up on authenticated users view Only of this condition is met, attempt to fix this!!

@tuxology
Copy link
Member

tuxology commented Mar 7, 2023

@khallow7 Can you file a separate issue regarding the correct ordering of "only few zubhub users" and "for all registered users" int he list? That is a valid bug

@khallow7
Copy link

khallow7 commented Mar 8, 2023

Will do that.

@srish srish changed the title Private projects are getting displayed publicly Draft projects should not show up in the search bar for user who created it Sep 1, 2023
@srish srish added good first issue Good for newcomers low priority bug Something isn't working labels Sep 1, 2023
@iykat
Copy link

iykat commented Sep 30, 2023

@tuxology @srish the title of the issue is a bit confusing.

the title says draft projects should not appear in a search result for the user who created them. however, @tuxology's comment here ("draft projects of other users ending up on authenticated users view") indicates it's only a problem when it appears for other users, and in that instance alone one should attempt to fix the problem.

These are two completely different issues in my understanding. can you kindly provide some clarity?

@iykat
Copy link

iykat commented Sep 30, 2023

I attempted to reproduce the issue by creating two different accounts. I realized that draft projects do indeed appear for the user who created them but do not appear for other users.

@srish
Copy link
Member

srish commented Oct 2, 2023

@iykat You can ignore @tuxology's comment as that issue got fixed earlier. The problem right now is exactly what is in the issue title: draft projects appear in a search result for the user who created them and they shouldn't.

@benndip
Copy link
Contributor

benndip commented Oct 3, 2023

Hello @srish , could I be assigned this issue? I have already solved it, and it works, I just want to be assigned so that I can open a PR for it ASAP.

Thanks!

@yokwejuste
Copy link
Collaborator

@srish found out what happened and will like to work on this and fix it

@benndip
Copy link
Contributor

benndip commented Oct 4, 2023

@srish @tuxology I just open a PR https://github.com/unstructuredstudio/zubhub/pull/775 that fixes this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers low priority
Projects
None yet
7 participants