Development #40802
permettre le login sans idp
0%
Description
Il n'y a pas de template, comme c'est le cas sur les autres briques.
Internal Server Error: /login/
Traceback (most recent call last):
...
File "/home/nroche/envs/publik-env/local/lib/python2.7/site-packages/django/template/loader.py", line 53, in select_template
raise TemplateDoesNotExist(', '.join(template_name_list), chain=chain)
TemplateDoesNotExist: registration/login.html
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-templates-add-a-login-template.patch 0001-templates-add-a-login-template.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
- bijoe pour le code et le template
- hobo pour les tests
Mis à jour par Frédéric Péters il y a environ 4 ans
Je ne vois pas trop l'intérêt à ajouter du code qui ne sera pas utilisé, juste pour pouvoir le tester.
Parce que là on va dans six mois prendre au sérieux ce gabarit, se demander pourquoi il utiliser <input type=submit> plutôt que <button> alors que tout aurait dû être converti, pourquoi il y a un bout dedans qui n'est pas traduit, etc.
~~
Maintenant, oui il peut y avoir une explication valable, qui me semble pouvoir être « avoir une page de login local va permettre de faire les tests en utilisant webtest, comme les autres briques, plutôt que le DjangoTestClient » et en ça je dirais qu'alors, oui, ok. Mais en corrigeant quand même le gabarit pour avoir traduction et <button> histoire qu'il ne vienne pas nous perturber plus tard.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-templates-add-a-login-template-40802.patch 0001-templates-add-a-login-template-40802.patch ajouté
permettre de faire les tests en utilisant webtest
c'est gentil, moi j'étais à mille lieux de penser à ça.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5e731e5b593141dd98952e013a6d6eff0e995abc
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Wed Mar 18 09:05:14 2020 +0100
templates: add a login template (#40802)
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Je note ici que django-webtest propose une méthode set_user() dans sa classe DjangoTestApp qui permet de ne pas avoir à utiliser de vue de login.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templates: add a login template (#40802)