Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Erreur 500 à la création d'une galerie avec un slug vide #6311

Merged
merged 1 commit into from
May 22, 2022

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented May 22, 2022

Fix #6294 .

Cette PR ajoute une alerte lorsqu'un utilisateur essaie de créer une galerie qui mènerait à un slug vide (et donc, une galerie incorrecte), par exemple en utilisant un titre tel que ":".
J'ai réutilisé ce qui est utilisé pour les titres de contenu, que j'ai extrait dans un fichier du module utils. Ca fonctionne bien mais je ne suis pas à 100% certain que le fichier que j'utilise (validators.py) est approprié.

Contrôle qualité

Scénario 1:

  • Se connecter
  • Ouvrir sa galerie d'images et cliquer sur "Créer une galerie"
  • Utiliser la valeur ":" (ou toute autre valeur qu'on filtre du slug) pour le titre
  • Soumettez le formulaire
    Résultat attendu: Un message d'erreur vous fait savoir que ce titre est invalide

Scénario 2:

  • Se connecter
  • Ouvrir sa galerie d'images et cliquer sur "Créer une galerie"
  • Utiliser la valeur ":e" (càd une combinaison de caractères valides et non-valides pour le slug) pour le titre
  • Soumettez le formulaire
    Résultat attendu: Le formulaire est correctement soumis, aucun message d'erreur n'est affiché

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 87.532% when pulling 7ab24da on Migwel:issue6294 into 3a3e3bc on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

@Arnaud-D Arnaud-D merged commit 36fdfd3 into zestedesavoir:dev May 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Erreur 500 à la création d'une galerie avec un slug vide
3 participants