Development #19614
ajout du support de Django 1.11
0%
Description
Suite à l'atelier de l'eocamp
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Fichier 0001-add-support-of-Django-1.11-19614.patch 0001-add-support-of-Django-1.11-19614.patch ajouté
- Patch proposed changé de Non à Oui
Ça fonctionne à l'exception des tests de hobo_notify:
_________________________________________________________________________________________________ ERROR collecting tests/test_hobo_notify.py __________________________________________________________________________________________________
tests/test_hobo_notify.py:6: in <module>
from corbo.hobo_agent.management.commands.hobo_notify import Command
corbo/hobo_agent/management/commands/hobo_notify.py:19: in <module>
from hobo.agent.common.management.commands import hobo_notify
.tox/coverage-django111/local/lib/python2.7/site-packages/hobo/agent/common/management/commands/hobo_notify.py:29: in <module>
from hobo.multitenant.utils import provision_user_groups
.tox/coverage-django111/local/lib/python2.7/site-packages/hobo/multitenant/utils.py:7: in <module>
from hobo.agent.common.models import Role
.tox/coverage-django111/local/lib/python2.7/site-packages/hobo/agent/common/models.py:5: in <module>
class Role(Group):
.tox/coverage-django111/local/lib/python2.7/site-packages/django/db/models/base.py:118: in __new__
"INSTALLED_APPS." % (module, name)
E RuntimeError: Model class hobo.agent.common.models.Role doesn't declare an explicit app_label and isn't in an application in INSTALLED_APPS.
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Lié à Development #19652: django 1.11: app_label explicite pour les applications ajouté
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Fichier 0001-add-support-of-Django-1.11-19614.patch 0001-add-support-of-Django-1.11-19614.patch ajouté
Patch à jour, avec les adaptations nécessaires dans le settings, les urls, les vues.
Dans les tests, passage par "reverse" pour la résolution des urls.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ça m'irait que ces modifications sur les tests qui ne me semblent pas liées à Django 1.11 soient dans un commit venant en premier.
# compatibility with django<1.10 if hasattr(global_settings, 'TEMPLATE_CONTEXT_PROCESSORS'):
n'est pas correct; la forme de settings avec TEMPLATES elle fonctionne déjà totalement en Django 1.8.
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Fichier 0001-add-support-of-Django-1.11-19614.patch 0001-add-support-of-Django-1.11-19614.patch ajouté
Ok.
Mis à jour par Frédéric Péters il y a plus de 6 ans
À noter la partie TEMPLATES parallèlement gérée dans #21007. (le patch attaché ici ajoute aussi le context process "request").
Mis à jour par Serghei Mihai il y a presque 6 ans
- Fichier 0001-add-support-of-Django-1.11-19614.patch 0001-add-support-of-Django-1.11-19614.patch ajouté
Patch à jour sans l'ajout de "TEMPLATES"
Mis à jour par Anonyme il y a presque 6 ans
- Statut changé de Nouveau à Solution proposée
Serghei Mihai a écrit :
Patch à jour sans l'ajout de "TEMPLATES"
Je n'arrive pas à appliquer le patch, même en --3way
Peut-être que tu pourrais faire un petit rebase ?
error: l'information de sha1 est manquante ou inutilisable (corbo/urls.py)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de En cours à Solution proposée
Parce qu'on ne peut pas laisser Corbo bloquer des migrations vers Stretch.
J'ai rebasé/refait/corrigé ce patch, j'ai aussi corrigé #20959 par rapport à mon commentaire, j'ai finalement désactivé les tests hobo (parce que comme c'est mis ça râle sur un app_label manquant sur le modèle Tenant, que ça doit pouvoir se contourner mais pas aujourd'hui).
https://git.entrouvert.org/corbo.git/log/?h=wip/19614-django-111
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à changé de Serghei Mihai à Frédéric Péters
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 7b39ac2f0b9cfdb0edf1048e2d4fd0521337d0d2
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Jan 3 20:13:28 2019 +0100
tests: disable hobo tests for 1.11
commit 4cd434e63191c0e9ee562aaa7514e52fc41c8cbb
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Tue Oct 24 09:56:49 2017 +0200
add support of Django 1.11 (#19614)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add support of Django 1.11 (#19614)