Skip to content

Privacy level of a project #3422

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

Closed
stsewd opened this issue Dec 19, 2017 · 5 comments
Closed

Privacy level of a project #3422

stsewd opened this issue Dec 19, 2017 · 5 comments
Labels
Bug A bug

Comments

@stsewd
Copy link
Member

stsewd commented Dec 19, 2017

Details

While replicating #3396 I noted that if the privacy level of a project is set to private I can still see the docs (logout user of course). The levels public and protected works as expected.

Expected Result

A logout user or no admin user can't see a project with privacy level set to private.

I know this is a beta feature, then this level of privacy has not been implemented?

@RichardLitt RichardLitt added the Bug A bug label Dec 20, 2017
@stsewd
Copy link
Member Author

stsewd commented Dec 21, 2017

I just discovered that if the privacy level is set to private the builds aren't show, the docs are still accessible. I think this is a little confusing, can we put better docs on this?

@humitos
Copy link
Member

humitos commented Dec 21, 2017

How it should work? Private shouldn't show the project nor the docs?

@stsewd
Copy link
Member Author

stsewd commented Dec 21, 2017

Public and protected are clear to me, with private I was expecting that no one can see the docs, but just hidden the builds and the project is more reasonable.

Maybe add a note that no matter the level of privacy, the docs are still seen by everyone?

@RichardLitt
Copy link
Member

Maybe add a note that no matter the level of privacy, the docs are still seen by everyone?

That makes sense to me.

stsewd added a commit to stsewd/readthedocs.org that referenced this issue Dec 22, 2017
@humitos
Copy link
Member

humitos commented Dec 28, 2017

Closing this one in favor of #2663

Thanks @stsewd for taking care of this!

@humitos humitos closed this as completed Dec 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants