ytorg/Yotter

Allow admins to remove restrictions

Open

#29 ouverte le 6 sept. 2020

Voir sur GitHub
 (4 commentaires) (1 réaction) (0 assignés)Python (36 forks)batch import
enhancementgood first issueimportant

Métriques du dépôt

Stars
 (467 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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.

Guide contributeur

Direction de recherche
Le problème nécessite de modifier les modèles HTML listés (ytsearch.html, channel.html, user.html, base.html) pour afficher conditionnellement les boutons et les routes de menu de suivi/arrêt de suivi uniquement lorsque l'utilisateur est authentifié, en fonction de l'option de configuration `restrictPublicUsage`. Consultez les exemples déjà résolus pour référence. Utilisez `user.is authenticated` avec des instructions if/else dans Jinja2.
Stack technique
pythondjango
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonDjangoJinja2
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70