search

rclsilver / django-lemonldap

Django application to make users to be authenticated through LemonLDAP::NG
Apache License 2.0
1 stars 4 forks source link

Add init and call functions for Django 1.10+ #1

Closed Madko closed 7 years ago

Madko commented 7 years ago

yo, ça farht ? A priori avec ces modifs ça marche avec django 1.11.

rclsilver commented 7 years ago

ça va et toi ? tu as regardé si ça fonctionnait toujours avec les versions précédentes ?

Madko commented 7 years ago

Yep ça va.

Bien vu. Je pense pas que ça marche avec les vieilles versions (<1.10). Comment qu'on fait ?

Le lun. 17 juil. 2017 à 08:10, Thomas Bétrancourt notifications@github.com a écrit :

ça va et toi ? tu as regardé si ça fonctionnait toujours avec les versions précédentes ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rclsilver/django-lemonldap/pull/1#issuecomment-315674572, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKsuiGbSFs0Hyab22CSE7GGeh076Hp1ks5sOvrcgaJpZM4OYs0U .

Madko commented 7 years ago

En fait si ça se trouve ça marche, vu que j'ai fait qu'ajouter une fonction init et surtout call, tout le reste est comme avant. Donc au pire en version < 1.10 la fonction init est appelée forcément, mais pour pas grand chose, et la call ne sert pas. J'essaye de tester ce soir.

Si ça ne supporte plus les versions < 1.10, peut-être le plus simple est de faire un bump majeur de version sur le module en précisant que ça supporte django 1.10+.

Madko commented 7 years ago

En fait non. J'ai testé tel quel avec 1.9.13 et ça ne fonctionne pas.

rclsilver commented 7 years ago

On peut imaginer faire une branche "before-1.10" avant ton merge, et merger ton code sur master

On Wed, Jul 19, 2017 at 8:43 AM, Edouard Bourguignon < notifications@github.com> wrote:

En fait non. J'ai testé tel quel avec 1.9.13 et ça ne fonctionne pas.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/rclsilver/django-lemonldap/pull/1#issuecomment-316288210, or mute the thread https://github.com/notifications/unsubscribe-auth/AAxiYrd1AKaHdJdU6qgOskhqPGu9ad7Bks5sPaV_gaJpZM4OYs0U .

-- Thomas Bétrancourt Tél. : 06.34.04.75.36 Fax : 09.72.26.10.48

Madko commented 7 years ago

ça me parait le plus simple en effet. Surtout que ça apporte 0 fonctionnalité. Après ya l'histoire de module pypi, est-ce que la version actuelle est capable de tirer en dépendance un django <1.10 ? Et la nouvelle elle devra tirer >= à 1.10.