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

Add proper body messages to HTTP 400 #39

Open
conorato opened this issue Oct 20, 2020 · 0 comments
Open

Add proper body messages to HTTP 400 #39

conorato opened this issue Oct 20, 2020 · 0 comments
Labels
enhancement New feature or request server issue linked to the polydodo server app

Comments

@conorato
Copy link
Contributor

conorato commented Oct 20, 2020

  • Préciser entre quelle et quelle taille est-ce qu'on accepte le fichier

  • ajouter or doesn't fit with the provided file du commentaire dans le message. À la place de parler de Received file, on devrait parler de File tout simplement, car si on affiche cette erreur côté client, c'aura plus de sens. Ou peut-être This combination of file, stream start, bedtime and wakeup time does not work. est encore plus direct selon moi. Et comme mentionné dans l'autre commentaire, on devrait propager cet erreur dans le front-end.

C'est peut-être moins dans le scope de la PR, mais il ne faut pas oublier le requis 1.1.3.4 Le fichier de données téléversé peut être refusé, s’il ne respecte pas le format attendu ou si il n’y a pas assez de données. et 1.1.3.5 La validation doit aussi être effectué entre le fichier et les moment de coucher/réveil. Par exemple, il faut refuser un fichier si le moment de réveil - moment de coucher > temps total de l’enregistrement.. Dans ces cas-là on pourrait renvoyer un message d'erreur dans le body d'un HTTP 400 qu'on afficherait dans le alert du form dans le front-end.

Voir fichier backend/classification/exceptions.py

Originally posted by @WilliamHarvey97 in #32 (comment)

@WilliamHarvey97 WilliamHarvey97 added enhancement New feature or request server issue linked to the polydodo server app labels Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request server issue linked to the polydodo server app
Projects
None yet
Development

No branches or pull requests

2 participants