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

Don't show "build ideas" to unprivileged users #3430

Closed
stsewd opened this issue Dec 21, 2017 · 2 comments
Closed

Don't show "build ideas" to unprivileged users #3430

stsewd opened this issue Dec 21, 2017 · 2 comments
Labels
Good First Issue Good for new contributors Needed: patch A pull request is required

Comments

@stsewd
Copy link
Member

stsewd commented Dec 21, 2017

Details

If a build fail and a build idea is present, then it is show to admin and no-admin users. If no-admin users clicked, a 404 is given. For example: https://readthedocs.org/projects/tinamit/builds/6364471/ (Try reseting it).

Expected Result

Don't show admin actions to no admin users.

Is there a valid reason for show this to no-admin users or if safe to hide this?

@humitos
Copy link
Member

humitos commented Dec 21, 2017

This is something that I notice the other day and I forget to report.

I think there is no reason to show the tips for non project admin or "staff" (not sure if staff is the correct one, but the permission used for the core team). This way, the end user and the core team can see the same and it's easy to debug in case it's needed.

My thoughts.

@ericholscher
Copy link
Member

+1 on not showing it. Would love a PR for this!

https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/templates/builds/build_detail.html#L101 is the line to change.

https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/templates/builds/build_detail.html#L66 is the logic you'd want to check.

stsewd added a commit to stsewd/readthedocs.org that referenced this issue Dec 21, 2017
@RichardLitt RichardLitt added Good First Issue Good for new contributors Needed: patch A pull request is required labels Dec 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Good for new contributors Needed: patch A pull request is required
Projects
None yet
Development

No branches or pull requests

4 participants