search

SergeyDavidovich / OMSBlazor

Real-time Order Management System
https://stupeni.net
MIT License
1 stars 1 forks source link

Is "Stastic" class is Entity? #27

Closed BekAllaev closed 1 month ago

BekAllaev commented 3 months ago

So now we inherit "Stastic" classes from Entity<T>. Is this entity or is this value object or maybe this is something else? Should "Stastic" classes be inside domain or somewhere else?

https://github.com/SergeyDavidovich/OMSBlazor/blob/0f56344b0133060301c9f8b0af715e3fc18641fd/src/OMSBlazor.Domain/Northwind/Stastics/OrdersByCountry.cs#L11

BekAllaev commented 1 month ago

Discussed with @SergeyDavidovich. We made a conclusion that stastic is DTO and it should not have ID. However at the end I have decided to inherit it from Entity<T> class of ABP so we can use it with IRepository from ABP and because I had problems with saving and updating record of stastics because EF Core ask for primary key for this operations but I dont use this ID anywhere more important is unique index of Key and TenantId