Projet

Général

Profil

Development #20098

newsletters: envoyer le numéro de mobile de l'usager

Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
15 novembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour les abonnements par SMS


Fichiers


Demandes liées

Lié à Corbo - Development #19846: faire evoluer l'API pour la prise en compte des abonnements par SMSFermé02 novembre 2017

Actions

Révisions associées

Révision c09f344d (diff)
Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

newsletters: send user mobile phone when available (#20098)

Historique

#1

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

  • Lié à Development #19846: faire evoluer l'API pour la prise en compte des abonnements par SMS ajouté
#2

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

En le récuperation depuis la session mellon.

#3

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

  • Statut changé de Nouveau à En cours
        if hasattr(self.user, 'saml_identifiers') and self.user.saml_identifiers.exists():
            self.params['uuid'] = self.user.saml_identifiers.first().name_id
+        else:
+            self.params['uuid'] = self.user.username

Pourquoi ? Si jamais il était décidé de prendre l'username comme uuid, cela ne pourraitpas être systématique ?

#4

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

Pour gérer l'existant, ou l'uuid des souscriptions est sur 32 caractères.
Or le username posé par l'agent hobo est coupé à 30 caractères.

Pour utiliser username il faudrait modifier le hobo_notify de corbo ainsi que faire une migration des données qui modifie les uuid de tous les abonnements à 30 caractères.

#5

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

Pour gérer l'existant, ou l'uuid des souscriptions est sur 32 caractères.

Désolé je ne pige pas le rapport avec le téléphone mobile.

#6

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

  • Fichier 0001-newsletters-send-user-mobile-phone-when-available-20.patch ajouté

Y en a pas.
j'ai fait un ticket dédié pour ça #20114.

#7

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

  • Fichier 0001-newsletters-send-user-mobile-phone-when-available-20.patch supprimé
#9

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

Je ne pige pas l'introduction de l'utilisation du username comme uuid ici. Ok ça a disparu dans le dernier patch, désolé du bruit.

#10

Mis à jour par Josué Kouka il y a plus de 6 ans

Ok

#11

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit c09f344d35b1729787b696c18073a0fb6dc5b2d2 (origin/master)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Wed Nov 15 18:30:27 2017 +0100

    newsletters: send user mobile phone when available (#20098)
#12

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 6 ans

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

Formats disponibles : Atom PDF