-
Notifications
You must be signed in to change notification settings - Fork 100
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 support for custom DTO #906
Comments
How would you see this happen in practice? There would need to be a specification of the mapping between the DTO and the entity type. For example, you would likely want your You could make a utility function in your library that does both DTO instantiation, and argument mapping for you. I think the
You can inspect the produced Let me know whether I understood your question correctly. If not, please show what you mean with a couple of API request/query/response snippets and some example DTO and entities. |
@Helveg that's a good question. There would need to be a mapping between DTO and entity fields. This would allow for exposing DTO fields for search, while querying the DB based on the mapped fields. I need to think about this more. We have ultimately built this in a custom fashion following: https://pietrzakadrian.com/blog/how-to-create-pagination-in-nestjs-with-typeorm-swagger. This lacks a lot of the user-facing query options available in nestjs-paginate. |
My entities and DTOs are separated to ensure that the API representation and DB representation can evolve independently. nestjs-paginate works well for querying, but assumes the entity and DTO are the same.
Is there any desire to support a second step after querying—DTO instantiation—to support separating these concerns?
The text was updated successfully, but these errors were encountered: