-
Notifications
You must be signed in to change notification settings - Fork 69
Open
Description
There are a lot of problems with this solution. The biggest and most glaring being the dependency on Infrastructure from Application through IDocumentContext (and there's even raw SQL statements in Application -- YIKES). This could have been and should be easily fixed by moving IDocumentContext to the Application project -- i.e. Dependency Inversion 101 and basically the underlying principle of "Clean Architecture."
It seems like there was a lot of work put into this code base and it's a pity this oversight renders it a very poor example of Clean Architecture.
Metadata
Metadata
Assignees
Labels
No labels