Développement #69228
profil utilisateur : arrêter de créer par défaut l’attribut de profil "numéro de téléphone"
0%
Description
Files
Related issues
Associated revisions
History
Updated by Paul Marillonnet about 2 years ago
- Related to Développement #49212: Création de compte avec un numéro de téléphone mobile added
Updated by Paul Marillonnet about 2 years ago
- File 0001-profile-remove-redundant-mobile-phone-field-69228.patch 0001-profile-remove-redundant-mobile-phone-field-69228.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Paul Marillonnet about 2 years ago
- Assignee set to Paul Marillonnet
(Et oublié de m’assigner le ticket.)
Updated by A. B. about 2 years ago
À 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.
Updated by Paul Marillonnet about 2 years ago
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).
Updated by A. B. about 2 years ago
- Status changed from Solution proposée to 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…
Updated by Paul Marillonnet about 2 years ago
Updated by Benjamin Dauvergne about 2 years ago
Si le seul changement c'est de ne plus créer le champ mobile par défaut, ça peut passer tout de suite.
Updated by Paul Marillonnet about 2 years ago
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.
Updated by Paul Marillonnet about 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique about 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
profile: remove redundant mobile phone field (#69228)