pix icon indicating copy to clipboard operation
pix copied to clipboard

[FEATURE] Enlever les espaces en trop quand on crée un profil cible(PIX-5405)

Open yaelle6 opened this issue 2 years ago • 8 comments

:unicorn: Problème

Lors de création d'un profil cible, on pouvait par erreur ajouter des espaces à la fin ou au début du nom, espaces qui étaient conservés en BDD. Ceci posait problème quand on souhaitait rechercher le profil cible car on recherchait pas avec le bon nom (qui contient des espaces).

:robot: Solution

Ajouter des méthodes qui clean les champs libres afin de ne plus avoir d'espaces indésirables

:rainbow: Remarques

Cette méthode a aussi été ajoutée pour certains champs de la création d'organisation.

:100: Pour tester

  • Se connecter à Pix Admin
  • Créer un profil cible avec des espaces et voir qu'ils n'ont pas été sauvegardés en BDD
  • Faire de même pour la création de l'organisation

yaelle6 avatar Aug 01 '22 16:08 yaelle6

I'm deploying this PR to these urls:

  • App (.fr): https://app-pr4731.review.pix.fr
  • App (.org): https://app-pr4731.review.pix.org
  • Orga: https://orga-pr4731.review.pix.fr
  • Certif: https://certif-pr4731.review.pix.fr
  • Admin: https://admin-pr4731.review.pix.fr
  • API: https://api-pr4731.review.pix.fr/api/

Please check it out!

pix-service avatar Aug 01 '22 16:08 pix-service

  • model.description = this.form.description ? this.form.description.trim() : null;

c'est bon c'est corrigé

yaelle6 avatar Aug 02 '22 16:08 yaelle6

En testant le formulaire de création de profil cible j'ai rencontré l'erreur suivante :

detail: "\"data.attributes.owner-organization-id\" must be a string"
status: "400"
title: "Bad Request"

image

frinyvonnick avatar Aug 03 '22 08:08 frinyvonnick

En testant le formulaire de création de profil cible j'ai rencontré l'erreur suivante :

detail: "\"data.attributes.owner-organization-id\" must be a string"
status: "400"
title: "Bad Request"

image

c'est corrigé

yaelle6 avatar Aug 04 '22 14:08 yaelle6

Les erreurs renvoyées du back ne sont pas affichée côté front (ici erreur sur le format de l'url) c'était déjà le cas avant ? image

lisequesnel avatar Aug 08 '22 08:08 lisequesnel

l'édition d'une campagne n'a pas enlevé les espaces avant le nom image

alors ça ne l'enlève pas dans le front mais ce qui est envoyé au back est sans espace. et si tu recharges la page, les espaces ne sont plus là. C'est coté Ember qui garde la version avec espace tant que tu ne recharge pas la page

yaelle6 avatar Aug 08 '22 12:08 yaelle6

func et teck ok, modulo mon dernier commentaire sur les crédits : https://github.com/1024pix/pix/pull/4731#discussion_r941248079

lisequesnel avatar Aug 09 '22 12:08 lisequesnel

func et teck ok, modulo mon dernier commentaire sur les crédits : #4731 (comment)

done ✅

yaelle6 avatar Aug 09 '22 12:08 yaelle6