Development #69228
profil utilisateur : arrêter de créer par défaut l’attribut de profil "numéro de téléphone"
0%
Description
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus d'un an
- Lié à Development #49212: Création de compte avec un numéro de téléphone mobile ajouté
Mis à jour par Paul Marillonnet il y a plus d'un an
- Fichier 0001-profile-remove-redundant-mobile-phone-field-69228.patch 0001-profile-remove-redundant-mobile-phone-field-69228.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Une piste qui nécessite au préalable une inspection de l’usage des champs Mobile et Téléphone dans Publik.
Rares sont les sites que j’utilise qui proposent encore les deux, ici on ne garderait qu’un seul champ Téléphone qui pourrait prendre le numéro de mobile de l’usager.
Les instances déployées avec un champ Mobile resteraient telles quelles, et dans un autre ticket, si l’inspection le valide, on écrirait la migration qui permet de ne conserver que le champ Téléphone pour ces instances déjà existantes.
Mis à jour par Paul Marillonnet il y a plus d'un an
- Assigné à mis à Paul Marillonnet
(Et oublié de m’assigner le ticket.)
Mis à jour par A. Berriot il y a plus d'un an
À la relecture ça me semble okay, juste cette assertion qui me parait étrange:
avant:
assert Attribute.objects.filter(name='mobile', kind='string').count() == 0
assert Attribute.objects.filter(name='mobile', kind='phone_number').count() == 1
après:
assert Attribute.objects.filter(name='phone', kind='phone_number').count() == 0
assert Attribute.objects.filter(name='phone', kind='string').count() == 1
Dans ma tête, on devrait avoir assert Attribute.objects.filter(name='phone', kind='phone_number').count() == 1, pas 0, mais les tests ont l'air de passer côté Jenkins, c'est étrange.
Mis à jour par Paul Marillonnet il y a plus d'un an
Agate Berriot a écrit :
Dans ma tête, on devrait avoir
assert Attribute.objects.filter(name='phone', kind='phone_number').count() == 1, pas 0, mais les tests ont l'air de passer côté Jenkins, c'est étrange.
Parce qu’on supprime le champ "mobile" de type "string", et qu’on teste maintenant la prise en compte du changement de type sur "phone". Ce champ passe maintenant du type "phone_number" à "string" (et non pas l’inverse pour le champ "mobile" dans la version originale des tests).
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Okay, merci pour l'explication, je pense qu'il me manquait le contexte du texte pour comprendre ce qui avait été changé dans le test (et redmine ne permet pas de facilement afficher les lignes précédentes / suivantes dans un patch). Je me basait sur ce qui était déclaré dans hobo/profile/migrations/0002_add_data.py donc forcément…
Mis à jour par Paul Marillonnet il y a plus d'un an
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Si le seul changement c'est de ne plus créer le champ mobile par défaut, ça peut passer tout de suite.
Mis à jour par Paul Marillonnet il y a plus d'un an
Benjamin Dauvergne a écrit :
Si le seul changement c'est de ne plus créer le champ mobile par défaut, ça peut passer tout de suite.
Oui complètement. Un rebase pas automatique car du pyupgrade passé entre temps, j’attends que Jenkins soit vert et je pousse.
Mis à jour par Paul Marillonnet il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit c13cbfb21daa31e05c168dd1a80e10e6958a058e
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Mon Sep 19 15:43:24 2022 +0200
profile: remove redundant mobile phone field (#69228)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
profile: remove redundant mobile phone field (#69228)