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.

Sign up for GitHub

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

Jump to bottom

Améliore les messages d'erreur pour l'importation d'image #6208

Merged
merged 1 commit into from
Nov 18, 2021
Merged

Améliore les messages d'erreur pour l'importation d'image #6208

merged 1 commit into from
Nov 18, 2021

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Nov 16, 2021 •
edited
Loading

Améliore les messages d'erreur pour l'importation d'image

Sujet du forum où le bug a été reporté

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back

  • Ajout d'image

    • Aller dans une galerie
    • Cliquer sur "Ajouter une image"
    • Sélectionner une image invalide (par exemple un PDF)
    • Envoyer le formulaire
    • Vérifier que le bandeau orange « Le fichier « Bidule » n'est une image valide. » s'affiche

  • Mise à jour d'image

    • Aller dans une galerie
    • Aller sur une image
    • Sélectionner une image invalide (par exemple un PDF)
    • Envoyer le formulaire (bouton « Mettre à jour »)
    • Vérifier que le bandeau orange « Le fichier « Bidule » n'est une image valide. » s'affiche
    • Vérifier que le message « Image invalide » s'affiche bien en dessous du bouton de sélection de l'image

@Situphen Situphen marked this pull request as ready for review November 16, 2021 18:18
@Situphen Situphen added the C-Back Concerne le back-end Django label Nov 16, 2021
@coveralls
Copy link

coveralls commented Nov 16, 2021 •
edited
Loading

Coverage Status

Coverage decreased (-0.02%) to 86.704% when pulling c7cc833 on Situphen:human-errors into 1c8e00b on zestedesavoir:dev.

@philippemilink
Copy link
Member

En attente de #6207 pour être QAisée et mergée.

philippemilink
philippemilink approved these changes Nov 18, 2021
View reviewed changes
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

OK ✔️

@philippemilink philippemilink enabled auto-merge (squash) November 18, 2021 21:01
@philippemilink philippemilink merged commit 5afa899 into zestedesavoir:dev Nov 18, 2021
@Situphen Situphen deleted the human-errors branch November 18, 2021 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-Back Concerne le back-end Django
Projects
Suivi des PR
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Situphen @coveralls @philippemilink