Add Provider #143
Add Provider #143
Conversation
|
|
||
| public string Email { get; set; } | ||
|
|
||
| public List<Product> OfferedProducts { get; set; } |
There was a problem hiding this comment.
En el DTO estas incluyendo una entity directo, que problemas te podría generar eso?
| public Provider Post([FromBody] ProviderDTO value) | ||
| { | ||
| TryValidateModel(value); | ||
| var provider = service.Create(mapper.Map<Provider>(value)); |
There was a problem hiding this comment.
Te falto descomentar el mappeo del Provider-ProviderDTO, por lo que todos los mappeos que se realicen van a fallar. Esta bueno que más allá de los test automáticos, se haga por lo menos una prueba básica de los nuevos endpoints, verificando que hagan lo que dicen hacer. Swagger UI es una interfaz amigable para probarlos.
| /// <param name="id">provider id to edit.</param> | ||
| /// <param name="value">provider information.</param> | ||
| [HttpPut("{id}")] | ||
| public void Put(string id, [FromBody] ProviderDTO value) |
There was a problem hiding this comment.
En el caso de que ocurra un error, se te ocurre alguna forma de manejar la respuesta para el consumidor de la API? LINK
Que controles en este caso podrías hacer para evitar una excepción?
Además estaría bueno dejar guardada información en algun lado si surge un error.
Description
Aquí deberías agregar un resumen del trabajo que realizaste.
Issues
Aquí deberías contar si tuviste algún problema o en el caso de que reportaste un issue.
Ej:
Related to #(issue)
Fixes # (issue)
Opinion
Si querés dejar algún comentario al respecto: improvements, mistakes, weaknesses.