Projet

Général

Profil

Development #65330

Ne pas enregistrer l'erreur "Impossible d’envoyer le SMS" quand le tel est un fixe ou pas renseigné

Ajouté par Anaïs Ecuvillon il y a presque 2 ans. Mis à jour il y a 9 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 mai 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Coup sur coup, pour 2 clients différents, je constate des erreurs "Impossible d'envoyer le SMS", par ex. ici :
https://formulaires.demarches.berreletang.fr/backoffice/workflows/2/logged-errors/5/
2 cas constatés :
  • le téléphone n'est pas un portable mais un fixe
  • il n'y a pas de tel renseigné (saisie en backoffice, tel optionnel)

Le connecteur en question est paramétré pour accepter uniquement "France métropolitaine (+33 [67])"
https://passerelle.demarches.berreletang.fr/ovh/sms_via_ovh_eo/

Il n'y a pas d'erreur d'un point de vue fonctionnel, c'est normal que le SMS ne partent pas dans les deux cas identifiés, est-ce qu'on ne pourrait pas ne pas enregistrer cette erreur ?

Révisions associées

Révision b28aaaa0 (diff)
Ajouté par Frédéric Péters il y a 9 mois

sms: do not record errors for application-level errors (#65330)

Historique

#2

Mis à jour par Nicolas Roche il y a presque 2 ans

- le téléphone n'est pas un portable mais un fixe
- il n'y a pas de tel renseigné (saisie en backoffice, tel optionnel)

Peut-être essayer d'ajouter ça dans la Condition d’exécution de l’action, pour que ce soit explicite ?
https://formulaires.demarches.berreletang.fr/backoffice/workflows/2/status/2/items/2/
https://demarches-hautes-pyrenees.test.entrouvert.org/backoffice/workflows/29/status/2/items/2/

#3

Mis à jour par Anaïs Ecuvillon il y a presque 2 ans

sur d'ancien formulaire, j'avais effectivement la condition :

form_var_telephone|slice:":2" == "06" or form_var_telephone|slice:":2" == "07" 

Mais dans une discussion récente avec ThomasN, j'ai cru comprendre que je n'en avais pas besoin vu le paramétrage du connecteur (mais j'ai sans doute fait un raccourci).

#4

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de En cours à Solution proposée
#6

Mis à jour par Robot Gitea il y a 9 mois

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

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a 9 mois

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

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Transition automatique il y a 9 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a 7 mois

Automatic expiration

Formats disponibles : Atom PDF