pytition / Pytition

Django app for self-hosted privacy-friendly online petitions
https://pytition.org
BSD 3-Clause "New" or "Revised" License
97 stars 28 forks source link

Fixbug #279

Closed bbmt-bbmt closed 3 years ago

bbmt-bbmt commented 3 years ago

Après avoir mis à jour mon instance, je me suis aperçu que maintenance mode attendait un fichier qui n'existait pas et qui a planté l'instance et que certaines commandes mode-maintenance dans la doc étaient erronées.

Amicalement

fallen commented 3 years ago

En tout cas si je suis l'installation docker sur un clone tout neuf ça a l'air de marcher : https://pytition.readthedocs.io/en/latest/installation.html#installation-via-docker-recommended-for-development

fallen commented 3 years ago

Au passage j'ai corrigé un bug dans le package django-maintenance-mode, le correctif est inclu dans la dernière release 0.16.0 : https://github.com/fabiocaccamo/django-maintenance-mode/blob/master/CHANGELOG.md Peut-être qu'il faut mettre à jour tes dépendances pip ? Dis moi s'il y a toujours un souci après ça :)

bbmt-bbmt commented 3 years ago

Désolé pour le délai de réponse, j'étais sur un autre projet et je manque un peu de temps. L'erreur que j'ai eu n'était pas sur une install neuve mais plutôt sur une upgrade 2.3 -> master. J'essaie de reproduire l'erreur dès que j'ai un peu de temps. Amicalement

fallen commented 3 years ago

Pas de souci, on manque tous de temps, je comprends :) Merci pour ton aide !

bbmt-bbmt commented 3 years ago

Je viens de re-tester en supprimant volontairement le fichier maintenance_mode_state.txt. Apparemment la nouvelle version de django-maintenance-mode le créé automatiquement lorsque il n'existe pas. Il n'y donc plus de bug.

Amicalement

fallen commented 3 years ago

Ok, top ! Merci pour le test :) Si tu peux supprimer le fichier de la pull-request, je merge tout ça :)

bbmt-bbmt commented 3 years ago

c'est fait Amicalement

fallen commented 3 years ago

Merci, je merge ! :)