Project

General

Profile

Développement #19614

ajout du support de Django 1.11

Added by Serghei Mihai about 7 years ago. Updated almost 6 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Frédéric Péters
Target version:
-
Start date:
24 October 2017
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Suite à l'atelier de l'eocamp

Files

0001-add-support-of-Django-1.11-19614.patch (1.93 KB) 0001-add-support-of-Django-1.11-19614.patch Serghei Mihai, 25 October 2017 03:42 PM
0001-add-support-of-Django-1.11-19614.patch (20.2 KB) 0001-add-support-of-Django-1.11-19614.patch Serghei Mihai, 05 January 2018 02:50 PM
0001-add-support-of-Django-1.11-19614.patch (19.9 KB) 0001-add-support-of-Django-1.11-19614.patch Serghei Mihai, 05 January 2018 03:15 PM
0001-add-support-of-Django-1.11-19614.patch (18.5 KB) 0001-add-support-of-Django-1.11-19614.patch Serghei Mihai, 25 May 2018 01:45 PM

Related issues

Related to Hobo - Développement #19652: django 1.11: app_label explicite pour les applicationsFermé25 October 2017

Actions

Associated revisions

Revision 4cd434e6 (diff)
Added by Serghei Mihai almost 6 years ago

add support of Django 1.11 (#19614)

History

#1

Updated by Serghei Mihai about 7 years ago

Ç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.

#2

Updated by Serghei Mihai about 7 years ago

#3

Updated by Serghei Mihai almost 7 years ago

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.

#4

Updated by Frédéric Péters almost 7 years ago

Ç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.

#6

Updated by Frédéric Péters almost 7 years ago

À noter la partie TEMPLATES parallèlement gérée dans #21007. (le patch attaché ici ajoute aussi le context process "request").

#9

Updated by Anonymous over 6 years ago

  • Status changed from Nouveau to 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)

#10

Updated by Benjamin Dauvergne over 6 years ago

  • Status changed from Solution proposée to En cours
#11

Updated by Serghei Mihai over 6 years ago

Il faut appliquer (et relire) #20959 d'abord.

#12

Updated by Frédéric Péters almost 6 years ago

  • Status changed from En cours to 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

#13

Updated by Serghei Mihai almost 6 years ago

  • Status changed from Solution proposée to Solution validée
  • Assignee changed from Serghei Mihai to Frédéric Péters
#14

Updated by Frédéric Péters almost 6 years ago

  • Status changed from Solution validée to 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)
#15

Updated by Frédéric Péters almost 6 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF