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

Show "access denied" when visiting a private/internal video #4768

Closed
kontrollanten opened this issue Feb 4, 2022 · 2 comments
Closed

Show "access denied" when visiting a private/internal video #4768

kontrollanten opened this issue Feb 4, 2022 · 2 comments
Labels
Type: Feature Request ✨ UI non-trivial UI changes, that might need discussion

Comments

@kontrollanten
Copy link
Contributor

Describe the problem to be solved
When a user is visiting a watch page of a video that's private/internal, the user will be redirected to login form if it's not logged in. This may be confusing for the user since it's not clear why they're going to the login page.

Describe the solution you would like:
It would be better if an access denied message would be shown, with a link to the login page.

@Chocobozzz
Copy link
Owner

What about a message above the login form? ("You need to be connected to access this resource")

@Chocobozzz Chocobozzz added Status: Waiting for answer Waiting issue author answer Type: Feature Request ✨ UI non-trivial UI changes, that might need discussion labels Feb 7, 2022
@kontrollanten
Copy link
Contributor Author

That would work. But I'd prefer something more user friendly; "You need to be logged in to see this video".

@Chocobozzz Chocobozzz removed the Status: Waiting for answer Waiting issue author answer label Feb 9, 2022
kontrollanten added a commit to kontrollanten/PeerTube that referenced this issue Jul 4, 2022
Chocobozzz pushed a commit to kontrollanten/PeerTube that referenced this issue Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Request ✨ UI non-trivial UI changes, that might need discussion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants