pix icon indicating copy to clipboard operation
pix copied to clipboard

[FEATURE] Permettre la modification des sujets d'un profil cible non relié à une campagne (PIX-12436).

Open Jeyffrey opened this issue 9 months ago • 2 comments

: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

  1. La méthode du TargetProfileSerializer deserializeCreationCommand a été renommée deserialize car elle est maintenant aussi utilisée pour la modification.

  2. Dans tous les cas, on ne permet pas la modification de la propriété isPublic.

  3. 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.

Jeyffrey avatar May 06 '24 09:05 Jeyffrey

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

pix-bot-github avatar May 06 '24 09:05 pix-bot-github

Revue fonctionnelle : Je ne peux pas créer de nouveau profil cible : image

Revue tech : Cela me semble bon 👌 Merci pour tous les changements opérés 🙇

frinyvonnick avatar May 22 '24 16:05 frinyvonnick