Project

General

Profile

Development #36331

compat django 2.2

Added by Frédéric Péters 11 months ago. Updated 8 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
22 Sep 2019
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

Description

De l'avant.


Related issues

Blocked by Hobo - Development #36584: Support MIDDLEWARE nouveau style dans debian_config_common.py Solution déployée 01 Oct 2019

Associated revisions

Revision bd6dbe50 (diff)
Added by Frédéric Péters 8 months ago

misc: get reverse{_lazy} from new django.urls module (#36331)

Revision 7810f162 (diff)
Added by Frédéric Péters 8 months ago

misc: update decorated_includes() for new django patterns (#36331)

Revision fe7f3482 (diff)
Added by Frédéric Péters 8 months ago

misc: update login/logout to be class based views (#36331)

Revision 3432648a (diff)
Added by Frédéric Péters 8 months ago

tests: update API tests to work with latest django-webtest (#36331)

Revision 1db1486d (diff)
Added by Frédéric Péters 8 months ago

misc: add new required renderer argument to render() (#36331)

Revision 4f1761a1 (diff)
Added by Frédéric Péters 8 months ago

misc: remove usage of six (#36331)

Revision 63bff03a (diff)
Added by Frédéric Péters 8 months ago

misc: update from fractions.gcd to math.gcd (#36331)

Revision 9a31e83d (diff)
Added by Frédéric Péters 8 months ago

tox: enable tests against django 2.2 (#36331)

Revision 56b133c7 (diff)
Added by Frédéric Péters 8 months ago

misc: correctly reset current directory after translations (#36331)

Revision 95c09426 (diff)
Added by Frédéric Péters 8 months ago

tox: avoid django-mellon 1.6 (#36331)

Revision 5bc475c6 (diff)
Added by Frédéric Péters 8 months ago

tests: don't run migration tests with sqlite and newer django (#36331)

History

#1 Updated by Frédéric Péters 11 months ago

  • Status changed from Nouveau to Solution proposée

Série de commits, dans la branche wip/36331-django-2.2, https://jenkins2.entrouvert.org/job/chrono-wip/job/wip%252F36331-django-2.2/ , dans l'ordre :

  • 38dcb78 misc: add on_delete parameter to foreign keys (#36331)

C'est désormais obligatoire.

  • f84f573 misc: get reverse{_lazy} from new django.urls module (#36331)

reverse et reverse_lazy doivent désormais être importés depuis django.urls.

  • 617222e misc: update decorated_includes() for new django patterns (#36331)

Il y a eu du travail sur les pattern mais ça reste relativement clean et est en gros juste s/RegexURLPattern/URLPattern/, mais en passant je vire aussi RegexURLResolver qui pareil n'existe plus mais n'était de toute façon pas utilisé.

  • 23e4fd1 misc: refer to user.is_anonymous as a property (#36331)

Trivial.

  • 261bc99 misc: update MIDDLEWARE settings name (#36331)

Pareil trivial ici, et pour gérer ça côté hobo, #36335.

  • 034d6b4 misc: update login/logout to be class based views (#36331)

Pas particulièrement différent.

  • bf7ffaf tests: use .set() to set many-to-many value (#36331)

Désormais obligatoire.

  • 48456f8 tests: update API tests to work with latest django-webtest (#36331)

Le dernier django-webtest vient avec sa propre classe d'authentification pour django rest framework ce qui fait que par défaut sur ces appels on récupère des 401 et plus des 403.

  • 7c54c3f misc: add new required renderer argument to render() (#36331)

Sans commentaire.

  • 9060dc3 misc: remove usage of six (#36331)

Ici on commence à virer la prise en charge en Python 2.

  • d32cea5 misc: update from fractions.gcd to math.gcd (#36331)

Pareil.

  • f51c6a9 tox: enable tests against django 2.2 (#36331)

Les tests pour 2.2.

  • 7c9dd90 misc: correctly reset current directory after translations

Pas vraiment lié à tout le reste, juste un bug sur lequel je suis tombé à un moment où compilemessages échouait.

  • db8617c tox: avoid django-mellon 1.6

C'est une release à éviter.

  • d59be72 tests: don't run migration tests with sqlite and newer django

Il y a un truc de transaction/migration avec sqlite qui rend ces tests compliqués, après différentes tentatives de mocking de bouts genre enable_constraint_checking, plutôt je zappe. (cf https://code.djangoproject.com/ticket/30023)

#2 Updated by Benjamin Dauvergne 11 months ago

  • Status changed from Solution proposée to Solution validée

Frédéric Péters a écrit :

  • 38dcb78 misc: add on_delete parameter to foreign keys (#36331)

C'est désormais obligatoire.

ok.

  • f84f573 misc: get reverse{_lazy} from new django.urls module (#36331)

reverse et reverse_lazy doivent désormais être importés depuis django.urls.

ok.

  • 617222e misc: update decorated_includes() for new django patterns (#36331)

Il y a eu du travail sur les pattern mais ça reste relativement clean et est en gros juste s/RegexURLPattern/URLPattern/, mais en passant je vire aussi RegexURLResolver qui pareil n'existe plus mais n'était de toute façon pas utilisé.

ok.

  • 23e4fd1 misc: refer to user.is_anonymous as a property (#36331)

Trivial.

ok.

  • 261bc99 misc: update MIDDLEWARE settings name (#36331)

Pareil trivial ici, et pour gérer ça côté hobo, #36335.

Pour MIDDLEWARE il faut aussi le support dans debian_config_common.py, j'ai ouvert un ticket #36584.

  • 034d6b4 misc: update login/logout to be class based views (#36331)

Pas particulièrement différent.

ok.

  • bf7ffaf tests: use .set() to set many-to-many value (#36331)

Désormais obligatoire.

ok.

  • 48456f8 tests: update API tests to work with latest django-webtest (#36331)

Le dernier django-webtest vient avec sa propre classe d'authentification pour django rest framework ce qui fait que par défaut sur ces appels on récupère des 401 et plus des 403.

ok.

  • 7c54c3f misc: add new required renderer argument to render() (#36331)

Sans commentaire.

ok.

  • 9060dc3 misc: remove usage of six (#36331)

Ici on commence à virer la prise en charge en Python 2.

ok.

  • d32cea5 misc: update from fractions.gcd to math.gcd (#36331)

Pareil.

ok.

  • f51c6a9 tox: enable tests against django 2.2 (#36331)

Les tests pour 2.2.

ok.

  • 7c9dd90 misc: correctly reset current directory after translations

Pas vraiment lié à tout le reste, juste un bug sur lequel je suis tombé à un moment où compilemessages échouait.

ok.

  • db8617c tox: avoid django-mellon 1.6

C'est une release à éviter.

ok.

  • d59be72 tests: don't run migration tests with sqlite and newer django

Il y a un truc de transaction/migration avec sqlite qui rend ces tests compliqués, après différentes tentatives de mocking de bouts genre enable_constraint_checking, plutôt je zappe. (cf https://code.djangoproject.com/ticket/30023)

Il me semble qu'il faut utiliser transactional_db, dans une transaction sqlite je ne suis pas certain qu'on puisse faire tous les changements de schéma que l'on souhaite (en tout cas je ne pense pas que ce soit énormément testé coté Django1).

Je valide, juste attendre #36584.

1 https://code.djangoproject.com/ticket/29182, ce ticket est corrigé mais dans un commentaires ils pointent que lancer des migrations dans une transaction n'est jamais testé

#3 Updated by Benjamin Dauvergne 11 months ago

  • Blocked by Development #36584: Support MIDDLEWARE nouveau style dans debian_config_common.py added

#4 Updated by Frédéric Péters 8 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 5bc475c6634e735e58b8cb00d95045404ccd9e4f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 21:20:08 2019 +0200

    tests: don't run migration tests with sqlite and newer django (#36331)

commit 95c09426c9afcc8c52c6c28c2480ba7ac802291f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Sep 22 13:09:07 2019 +0200

    tox: avoid django-mellon 1.6 (#36331)

commit 56b133c7a7aff380843e4dd8b9528e2755072886
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Sep 22 13:07:58 2019 +0200

    misc: correctly reset current directory after translations (#36331)

commit 9a31e83d1081e292dd6e72ed696113792753c5aa
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 21:22:27 2019 +0200

    tox: enable tests against django 2.2 (#36331)

commit 63bff03a87b4dcfc991d964162dc08f8d64fe031
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 22:37:04 2019 +0200

    misc: update from fractions.gcd to math.gcd (#36331)

commit 4f1761a1a23e2346b9a8e0632a2f06d4edf9da28
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 22:36:08 2019 +0200

    misc: remove usage of six (#36331)

commit 1db1486d5874ab67268f2adfa5a378ab790df42d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 22:31:48 2019 +0200

    misc: add new required renderer argument to render() (#36331)

commit 3432648af790acc2c6ac051c136dff122a050ec6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 21:21:14 2019 +0200

    tests: update API tests to work with latest django-webtest (#36331)

commit fe7f3482c8a85048cb188a796195e5f52b055174
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 21:19:33 2019 +0200

    misc: update login/logout to be class based views (#36331)

commit 7810f162650e488587a235d6686f00d3e0d962f3
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 21:17:30 2019 +0200

    misc: update decorated_includes() for new django patterns (#36331)

commit bd6dbe50fd0777101ae2daea9c9129dfe7031726
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Sep 21 21:16:23 2019 +0200

    misc: get reverse{_lazy} from new django.urls module (#36331)

#5 Updated by Frédéric Péters 8 months ago

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

Also available in: Atom PDF