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.

Sign up for GitHub

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

Jump to bottom

SCL Data Service should support DTO's #357

Closed
pascalwilbrink opened this issue Oct 11, 2023 · 2 comments
Closed

SCL Data Service should support DTO's #357

pascalwilbrink opened this issue Oct 11, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@pascalwilbrink
Copy link
Member

pascalwilbrink commented Oct 11, 2023 •
edited by Sander3003
Loading

In order to make the implementation better to maintain; The idea is to replace existing custom code by standard implementation. In order to do so; the DTO is concept is needed to prevent errors.

If we want to migrate to native Repositories (like has been done in Sitipe), we need to support DTO's (Data Transfer Objects).

App > Service > Repository
DTO > DTO/Entity translation > Entity

The Input and Output of the Service package should be DTO's.
The translation from/to a DTO should be in the Service.
@pascalwilbrink pascalwilbrink added the enhancement New feature or request label Oct 11, 2023
@pascalwilbrink pascalwilbrink mentioned this issue Oct 11, 2023
Closed
@Sander3003
Copy link
Member

Can this be closed @pascalwilbrink ?

@Sander3003
Copy link
Member

Considered closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
CoMPAS Issues Overview Board
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pascalwilbrink @Sander3003