Skip to content

Comments

Valudation for DTO and useful response in case of error#7

Open
denis-tsv wants to merge 1 commit intovkhorikov:masterfrom
denis-tsv:master
Open

Valudation for DTO and useful response in case of error#7
denis-tsv wants to merge 1 commit intovkhorikov:masterfrom
denis-tsv:master

Conversation

@denis-tsv
Copy link

Now CustomersController.Create method returned not useful for API clients response: one error message for all DTO properties.
If we will use ModelState then we can solve two problems:

  1. We will receive useful response where each property has array or error messages
  2. If DTO property is not valid (for example too long) and it is not mapped to ValueObject then we also will receive an error message.

@vkhorikov
Copy link
Owner

I don't like ModelState because it produces json that is too verbose. It's only useful when you work with Razor views or some other built-in asp.net web views. In scenarios with rich UI clients (e.g written in React), I recommend returning custom DTOs.

You can gather multiple errors with the approach I described here: https://enterprisecraftsmanship.com/posts/combining-asp-net-core-attributes-with-value-objects/

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants