pix
pix copied to clipboard
[FEATURE] Permettre la modification des sujets d'un profil cible non relié à une campagne (PIX-12436).
:unicorn: Problème
Nous avons l'objectif de rendre possible la duplication d'un profil cible existant en autorisant la modification de son référentiel.
Or, à l'heure actuelle, il n'est pas permis de modifier ce référentiel.
:robot: Proposition
La route PATCH /api/admin/target-profiles/{id}
accepte maintenant les tubes
en payload.
Pour respecter le DDD, dans le modèle TargetProfileForAdmin
, on a ajouté une méthode dédiée à l'update
qui gère les besoins métier :
- on permet la modification des tubes uniquement si le profil cible n'est pas relié à une campagne,
- on vérifie la validité de la nouvelle valeur de la propriété
category
, - on vient mettre à jour les données du model directement dans cette méthode.
:rainbow: Remarques
-
La méthode du TargetProfileSerializer
deserializeCreationCommand
a été renomméedeserialize
car elle est maintenant aussi utilisée pour la modification. -
Dans tous les cas, on ne permet pas la modification de la propriété
isPublic
. -
Le
allowUnknown
(https://github.com/1024pix/pix/blob/dev/api/lib/application/target-profiles/index.js#L337) sur la payload a été supprimé, il convient donc de modifier l'adapter d'Ember.
:100: Pour tester
- Tests verts
- Vérifier que la modification de profil cible fonctionne toujours sur PixAdmin.
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants :
Revue fonctionnelle :
Je ne peux pas créer de nouveau profil cible :
Revue tech : Cela me semble bon 👌 Merci pour tous les changements opérés 🙇