Développement #69895
supprimer le modèle RoleAttribute
Start date:
05 October 2022
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
Migrer les données et en faire de vrais champs du modèle Role.
Files
Related issues
Associated revisions
History
Updated by Valentin Deniaud about 2 years ago
- Related to Développement #70672: agent authentic, adaptations pour la suppression du modèle RoleAttribute added
Updated by Valentin Deniaud about 2 years ago
- File 0001-a2_rbac-move-role-attributes-to-real-model-fields-69.patch 0001-a2_rbac-move-role-attributes-to-real-model-fields-69.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Ça se fait très bien.
Dans la foulée je supprime brutalement attributes_ng/sources/service_roles.py parce que je n'ai aucune idée d'à quoi ça sert, le seul test que ça casse est test_add_attributes_user_ou1_role_ou2 dont je n'arrive pas à voir à quelle situation réelle il correspond.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to Solution validée
Ok, vendredi ?
Updated by Benjamin Dauvergne about 2 years ago
Ah si une remarque, ne manquerait-il pas un blank=True sur le champs emails (je ne sais pas trop comment est traité la liste vide).
Updated by Valentin Deniaud about 2 years ago
Benjamin Dauvergne a écrit :
Ah si une remarque, ne manquerait-il pas un blank=True sur le champs emails (je ne sais pas trop comment est traité la liste vide).
Non pas besoin, j'ai ajouté un bout de test dans la branche.
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 5f0c03e32f67cf754972ebf02921f942d8f6df73
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Oct 25 14:27:52 2022 +0200
a2_rbac: move role attributes to real model fields (#69895)
Updated by Valentin Deniaud about 2 years ago
- Related to Développement #71844: Inclure les anciens attributs de rôle dans l'import/export added
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
a2_rbac: move role attributes to real model fields (#69895)