Projet

Général

Profil

Development #26716

iws: exposer le paramètre de notification par mail

Ajouté par Emmanuel Cazenave il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
25 septembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Un paramètre de notification par mail 'I_AP_EMAIL' à 'oui' ou 'non' est envoyé à IWS, il faut l'exposer directement dans les paramètres du connecteur.

Fichiers

0001-iws-expose-email-notification-parameter-26716.patch (4 ko) 0001-iws-expose-email-notification-parameter-26716.patch Emmanuel Cazenave, 25 septembre 2018 16:11

Révisions associées

Révision e2c0b653 (diff)
Ajouté par Emmanuel Cazenave il y a plus de 5 ans

iws: expose email notification parameter (#26716)

Historique

#2

Mis à jour par Emmanuel Cazenave il y a plus de 5 ans

#3

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

Juste des remarques pour montrer que j'ai bien lu :

  1. J'aime toujours pas les « truc = machin if bidule else chose » mais, alleÿy.
  2. En terme de "compatibilité", ça serait pas intéressant de laisser la valeur à "OUI" (ou "NON") s'il n'y a pas de "email_notif" ? (et donc le rendre non obligatoire).
  3. Pas besoin de faire pareil pour les SMS ?
#4

Mis à jour par Emmanuel Cazenave il y a plus de 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Thomas Noël a écrit :

Ack.

Juste des remarques pour montrer que j'ai bien lu :

  1. J'aime toujours pas les « truc = machin if bidule else chose » mais, alleÿy.

Merci, et moi je trouve ça très élégant, les goût est les couleurs ma petite dame ...

  1. En terme de "compatibilité", ça serait pas intéressant de laisser la valeur à "OUI" (ou "NON") s'il n'y a pas de "email_notif" ? (et donc le rendre non obligatoire).

Ils le veulent quoiqu'il arrive, donc bon paramètre obligatoire et allons y.

  1. Pas besoin de faire pareil pour les SMS ?

Excellente question, mais à priori non puisqu'ils n'ont pas soulevé ce point, bref je laisse ça pour l'instant.

commit e2c0b653c3a32dd8c06ea9b81c6cda6791083e80
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Tue Sep 25 16:10:05 2018 +0200

    iws: expose email notification parameter (#26716)
#5

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF