-
Notifications
You must be signed in to change notification settings - Fork 423
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
Issue in implementation according to CLEAN Arch #6
Comments
please when will this issue be close? |
@Kaaveh is very correct. Suggestion
The Advantage of this is that, when for example we change the cache mechanism (database), the changes will only take place in the data layer not changing the model in the domain layer |
@olukayodepaul, could you create a PR w/ the changes? |
@Patrulescu-Ronald-Sandrino here is the link to my repo where you can see the separation https://github.com/olukayodepaul/FaceOff |
In
data class Note
in the domain, you mark it as@Entity
with Room annotation. I think you involve your domain layer with data layer implementation and if you change your DB in future, your domain layer take effect.The text was updated successfully, but these errors were encountered: