pix
pix copied to clipboard
[FEATURE] Enlever les espaces en trop quand on crée un profil cible(PIX-5405)
: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
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!
- model.description = this.form.description ? this.form.description.trim() : null;
c'est bon c'est corrigé
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"
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"
c'est corrigé
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 ?
l'édition d'une campagne n'a pas enlevé les espaces avant le nom
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
func et teck ok, modulo mon dernier commentaire sur les crédits : https://github.com/1024pix/pix/pull/4731#discussion_r941248079