Enlever les HttpStatusCodeResult
Moustachauve opened this issue · 2 comments
Moustachauve commented
Description
Dans le panneau d'admin, plusieurs méthodes utilisent des "return new HttpStatusCodeResult" avec des codes d'erreurs. Ce n'est pas un WebApi, il faudrait donc gérer les erreurs correctement en redirigeant vers une page appropriée et en affichant les erreurs.
- Enlever tous les HttpStatusCodeResult
- Afficher un message d'erreur convenable à l'utilisateur
huideaki commented
@Moustachauve est-ce vraiment nécéssaire d'après ce que j'ai lu c'est quand même une bonne façon de faire que si l'id est null ou mauvais une redirection est fait de la bonne façon
Moustachauve commented
Ça peut être correcte de rediriger vers des erreurs http mais généralement c'est plus clean de retourner 200 quand même parce que la requête a bien marcher mais d'afficher un message au user (genre Le nom d'utilisateur est requis). C'est vraiment low priority.