vinopinions / api

1 stars 0 forks source link

Return the same relations everywhere #143

Closed devskar closed 8 months ago

devskar commented 9 months ago

We should use dto's everywhere in the controller. We might even should use @automapper/core instead of class transformer. This would make the process more clear but we should try that first

devskar commented 9 months ago

I am in favour of using @automapper because I believe that ultimately it will give us the most control and we can keep at least some decorators out of the entity files which will make the file at least a little bit cleaner.

devskar commented 9 months ago

Nevermind, we should not user @automapper because of https://github.com/nartc/mapper/issues/536. The project is moving very slow and there is no feedback on issues. Will have to keep looking

devskar commented 9 months ago

Not quite sure where to go from here without automapper

devskar commented 8 months ago

I believe this was kind of handled in #178