#54 Automatic deletion of expired contribs and renewal notifications

Implementation of #27. This is still just a work in progress. Will finish later.

Realy really happy: I finished this PR… And unfair data retention at the same time…

I'd like some reviewing/testing on it before we merge it.

@opi @vross @sileth @ljf got any time for a review ?

Bumpy-bump, quelqu'un·e pour une review/test ?

I can test (I'll try to do so during the week). o/

@vross trop cool :-)

Demain après-midi on fait du code à FAImaison. Si à tout hasard tu avais testé d'ici là, ça me permettrait de prendre en compte tes retours à ce moment là… Et éventuellement d'intégrer.

Et puis sinon tant pis, ça attendra plus tard :-).

En relecture sans test j'ai trouvé quelques typos :

really* & random*

• https://code.ffdn.org/FFDN/wifi-with-me/pulls/54/files#diff-8c52d1086430334cb2c5fe67779bfc73da7d7b7R135

1 space before text in docstring.

• https://code.ffdn.org/FFDN/wifi-with-me/pulls/54/files#diff-257d74ab9efb669d01cb694e437dd810822ad0aR1
• https://code.ffdn.org/FFDN/wifi-with-me/pulls/54/files#diff-57a7439762a7eebf6d071ea30d91a9c7455bc63R23
• https://code.ffdn.org/FFDN/wifi-with-me/pulls/54/files#diff-57a7439762a7eebf6d071ea30d91a9c7455bc63R201

grâce*

• https://code.ffdn.org/FFDN/wifi-with-me/pulls/54/files#diff-96cf0ebfd00cc84ffdfa7ec2935681f225bc5cdR9
• https://code.ffdn.org/FFDN/wifi-with-me/pulls/54/files#diff-b6a4bfcc5ea317612bd883ed6b1be97999a3ebfR7

Les deux commandes delete_expired_contribs et send_expiration_reminders fonctionnent correctement en --dry-run.

1 space before text in docstring.

C'est mal ? Parce-que c'est fait exprès en fait :-] (lisibilité, même logique qu'après un "#").

Sinon j'ai corrigé le reste, merci <3. Hop, reste à finir un test complète que tout marche comme ça devrait :-)

Du coup, j'ai testé vite fait :

• delete_expired_contribs fonctionne bien, ç'a supprimé uniquement ce qui est expiré :)

• send_expiration_reminders a l'air de bien marcher aussi ; comme mon système est en anglais, ça donne une formulation un peu bizarre ("Vous aviez déposé le March 5, 2017, 10:10 a.m. une demande") mais bon, on comprend ;)

Je sais pas pourquoi, je m'attendais à ce que l'interface d'admin django me prévienne que la contrib avait expiré mais en fait non.

Merci pour tout ce boulot en tous cas ! :)

comme mon système est en anglais, ça donne une formulation un peu bizarre ("Vous aviez déposé le March 5, 2017, 10:10 a.m. une demande") mais bon, on comprend ;)

Corrigé (j'en ai ch… et le fix me parait moyen propre, mais je n'ai vraiment pas trouvé mieux).

Je sais pas pourquoi, je m'attendais à ce que l'interface d'admin django me prévienne que la contrib avait expiré mais en fait non.

Corrigé.

Merci pour les tests / relectures.