dirigible icon indicating copy to clipboard operation
dirigible copied to clipboard

[Core] Add `TenantStatus` to `Tenant` interface

Open g051051 opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe. The Tenant interface does not provide the TenantStatus property from the Tenant entity. Without the TenantStatus property, the data is incomplete. This is important for implementing a Tenant view, along with create, update (especially enable/disable), and delete capabilities.

Describe the solution you'd like Add the TenantStatus field to the Tenant interface and any classes that implement it.

Describe alternatives you've considered The only alternative would be to not use Tenant interface based objects, or create a new interface that extends it with the TenantStatus property.

Additional context The reason for wanting this is because when reading the set of Tenant objects from the DB, we need to transform them into TenantImpl objects. Since TenantImpl implements Tenant, it doesn't have the TenantStatus.

g051051 avatar Apr 08 '24 14:04 g051051

@g051051 @ThuF is this issue still applicable?

iliyan-velichkov avatar Jan 21 '25 15:01 iliyan-velichkov

@iliyan-velichkov I don't see any indication that it's been changed. However, I'm not working on creating a Tenant view any longer, so I suppose it can be closed if no one else thinks it's necessary (or has an alternate way to accomplish what it was for).

g051051 avatar Jan 21 '25 15:01 g051051