Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Passage à Django 1.9 #3235

Closed
wants to merge 36 commits into from
Closed

[WIP] Passage à Django 1.9 #3235

wants to merge 36 commits into from

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented Dec 8, 2015

Voir #3245

SpaceFox and others added 30 commits November 11, 2015 16:16
Donc on supprime cette déclaration qui était de toutes façons inutile même avec Django 1.7.
Cf https://docs.djangoproject.com/en/1.8/releases/1.8/#management-commands
Impacts directs obligatoires :
- Une relation en base change de nom pour être python-compatible (et donc sa migration en base)
- On doit maintenant préciser le paramètre --fake-initial lors d'une migration de MAJ (cf https://docs.djangoproject.com/en/1.8/ref/django-admin/#django-admin-option---fake-initial)
- Debug Toolbar en v1.3
- Fix pour que oauth fonctionne, cf jazzband/django-oauth-toolkit#204
(compatibilité officielle à partir de 1.0.4)
@DevHugo
Copy link
Contributor

DevHugo commented Dec 8, 2015

J'en ajoute une:

  • Vérifier que notre version de Haystack supporte Django 1.9 …

Bonne chance pour la migration =).

@gustavi
Copy link
Contributor Author

gustavi commented Dec 9, 2015

Bon alors j'ai une erreur sur un template qui n'existe pas avec crispy donc si quelqu'un a une idée je prends car je sèche.

Pour reproduire : aller sur un page avec un formulaire utilisant crispy, la page de connexion par exemple.

Logs : http://paste.awesom.eu/ocoS

@GerardPaligot
Copy link
Member

Je n'ai pas trop le temps de regarder là mais il semblerait que ButtonHolder (et d'autres) puissent foutre un peu le boxons: http://stackoverflow.com/a/32564292/3006692

@gustavi
Copy link
Contributor Author

gustavi commented Dec 9, 2015

Il s'agit d'un template manquant dans la dépendance. PR en cours : django-crispy-forms/django-crispy-forms#551

@DevHugo
Copy link
Contributor

DevHugo commented Dec 9, 2015

Je suis pas sur que que tu obtienne une réponse rapidement, tu me diras, on peut attendre. Y'a pas un moyen de contourner le souci ?

@gustavi
Copy link
Contributor Author

gustavi commented Dec 9, 2015

La seule solution que je vois est de créer le template chez nous.

@DevHugo
Copy link
Contributor

DevHugo commented Dec 9, 2015

ça peut-être une bonne solution temporaire. SI c'est bloquant pour toi.

@gustavi
Copy link
Contributor Author

gustavi commented Dec 9, 2015

Pour le moment c'est pas bloquant dans la mesure où plusieurs dépendances sont pas encore compatibles. Si jamais il ne reste que crispy alors j'utiliserai cette solution.

@GerardPaligot
Copy link
Member

Concernant l'API, nous sommes en version 3.1.1 de DRF mais DRF supporte Django 1.9 que depuis sa version 3.3.0.

@gustavi
Copy link
Contributor Author

gustavi commented Dec 10, 2015

Ma PR utilise bien la dernière version.

@GerardPaligot
Copy link
Member

Je pensais que tu n'avais pas mis à jour la librairie ? ^^°

@gustavi
Copy link
Contributor Author

gustavi commented Dec 10, 2015

Bon alors les dernières nouvelles : j'ai trouvé d'où venait le bug niveau API avec REST (c'était plutôt caché et en fait un bug fait que les Warning pour ce qui est déprécié sur ce cas là ne s'affiche pas...) MAIS (bah oui sinon c'est pas drôle) on a django-auth-toolkit qui vient foutre son grain de sable.

En fait cette ligne là ne fonctionne pas https://github.com/gustavi/zds-site/blob/django19/zds/member/api/tests.py#L1004. Tout simplement je me mange un unsupported_grant_type sans aucune raison. J'ai cherché un bon moment sans trouvé, il est tard et je suis surement passé à côté donc je veux bien un peu d'aide sur ce point si quelqu'un a le courage (ping @artragis et @GerardPaligot qui connaissent un peu cette partie du code).

@artragis
Copy link
Member

pour les modules article et tutorial OSEF. Je comprends même pas qu'on n'ai pas encore mergé la pr qui les supprime.

@artragis
Copy link
Member

j'ai fait une PR sur ton dépôt

corrige les erreurs de tutorialv2 et munin
@DevHugo
Copy link
Contributor

DevHugo commented Dec 13, 2015

Je viens de regarder Haystack n'est pas compatible avec Django 1.9, c'est ce ticket qu'il faut regarder sur Haystack django-haystack/django-haystack#1277 pour voir la progression.

Il faut donc faire une montée de version de Haystack mais pour cela, il faut changer une bonnne partie de notre code car on était pas compatible avec les dernières versions.

@gustavi
Copy link
Contributor Author

gustavi commented Dec 13, 2015

Merci, je vais de modifier le premier post avec toutes les dépendances dont on attend une màj ou qui sont à vérifier.

Je rappelle que Django 1.7 n'est plus supporté.

@DevHugo
Copy link
Contributor

DevHugo commented Dec 15, 2015

Merci, je vais de modifier le premier post avec toutes les dépendances dont on attend une màj ou qui sont à vérifier.
Je rappelle que Django 1.7 n'est plus supporté.

Je suis entièrement d'accord avec toi, il faut monter la version de Django, le plus vite possible. Merci encore pour ton courage d'aller défier les changelog =)

@artragis
Copy link
Member

pour model-utils ce qui est étrange c'est que tout les problèmes ont été résolus dès qu'on est passé à la dernière version (cf ma pr) je pense donc que de leur côté ils n'ont qu'à vérifier en fait pour assurer le support.

@gustavi gustavi mentioned this pull request Dec 16, 2015
4 tasks
@gustavi
Copy link
Contributor Author

gustavi commented Dec 16, 2015

Voir #3245

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants