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

Enlever les HttpStatusCodeResult #31

Open
2 tasks
Moustachauve opened this issue Feb 6, 2016 · 2 comments
Open
2 tasks

Enlever les HttpStatusCodeResult #31

Moustachauve opened this issue Feb 6, 2016 · 2 comments

Comments

@Moustachauve
Copy link
Collaborator

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
Copy link
Contributor

@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
Copy link
Collaborator Author

Ç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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants