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.

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

Infrastructure.Persistence module does not utilize IEntityTypeConfiguration #9

Open
baratgabor opened this issue Feb 19, 2021 · 0 comments
Labels
backend enhancement New feature or request

Comments

@baratgabor
Copy link
Owner

The Clean Architecture Template utilizes IEntityTypeConfiguration to separate EF Core configuration on a per-entity basis.

This sample project does not utilize this technique, which makes this aspect of the design of the Persistence module arguably inferior. (The reason for this peculiar omission is that originally I didn't actually use the template, but started from an empty default solution, to force myself to engage more with the design instead of relying on a predefined structure.)

Task:

  • Spread out entity configurations from ApplicationDbContext into classes implementing IEntityTypeConfiguration.
  • Try to keep owned type configuration centralized in some way (since it seems currently all references to these types, e.g. Money, have to be configured individually, including their value conversion, even though this configuration is basically the same in all cases).
@baratgabor baratgabor added enhancement New feature or request backend labels Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant