ytorg/Yotter

Allow admins to remove restrictions

Open

#29 opened on Sep 6, 2020

View on GitHub
 (4 comments) (1 reaction) (0 assignees)Python (36 forks)batch import
enhancementgood first issueimportant

Repository metrics

Stars
 (467 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

As for what's explained here we should try to make the app work even if there's no authenticated user.

It can easily be done using the user.is_authenticated method and Jinja2 if/else statements.

(See the ones that are resolved for reference) When the config option restrictPublicUsage is False and the user is NOT logged in, this pages need:

  • ytsearch.html ⇾ Don't show follow/unfollow when not logged in.
  • channel.html ⇾ Don't show follow/unfollow when not logged in.
  • search.html ⇾ I think nothing is required. Test.
  • user.html ⇾ Don't show follow/unfollow when not logged in.
  • base.html ⇾ Show ONLY AVAILABLE routes on menu when restrictions are disabled.

Contributor guide

Research direction
Implement conditional display logic in Jinja2 templates based on `restrictPublicUsage` config and `user.is authenticated`.
Tech stack
pythondjango
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
PythonDjangoJinja2
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70