Projet

Général

Profil

Bug #22668

notification des factures : le premier message ne devrait pas être un rappel

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
21 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

J'ai toujours autant de mal à suivre la logique désirée.

            # invoice can be paid
            if pay_limit_date > now + remind_delta:
                message = _('Invoice %s to pay') % invoice['label']
            else:
                message = _('Reminder: invoice %s to pay') % invoice['label']
                notification_id = notification_reminder_id

Mais je viens peut-être de comprendre une des choses qui me chagrinaient : on peut se trouver à avoir comme premier et unique message un rappel, ce qui n'a pas de sens. (je trouve)

Fichiers

0001-lingo-send-reminder-notification-only-if-first-notif.patch (4,21 ko) 0001-lingo-send-reminder-notification-only-if-first-notif.patch Benjamin Dauvergne, 21 mars 2018 22:48

Historique

#2

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Voilà, je j'envoie le reminder que si la première notification a été acké.

M'enfin c'est cette histoire de reminder que je trouve débile. Ça revient exactement à dé-acker la première notification, parce qu'on va se retrouver par exemple à acker la première notification et si on se trouve dans la période de "remind" hop une nouvelle notification apparaîtra, selon quand le cron tournera, exactement ce que tu ne voulais pas (dé-acker une notification ou envoyer deux fois une notification pour un même évènement c'est pareil).

#3

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Nouveau à Rejeté

Le code a avancé sans ça.

Formats disponibles : Atom PDF