fullstackhero / dotnet-starter-kit

Production Grade Cloud-Ready .NET 8 Starter Kit (Web API + Blazor Client) with Multitenancy Support, and Clean/Modular Architecture that saves roughly 200+ Development Hours! All Batteries Included.

Home Page:https://fullstackhero.net/dotnet-webapi-boilerplate/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Is there any plan to migrate to Finbuckle 7.0.0 ?

bvgnet opened this issue · comments

Yes.

  1. This fails even in v2 branch because FinBuckle removed the getter from the MultiTenantContext property.

https://github.com/fullstackhero/dotnet-webapi-starter-kit/blob/2af2c4f6d8947aa63153d992fba48740ffda1d30/api/framework/Infrastructure/Tenant/Services/TenantService.cs#L64C1-L68C15

image

  1. The removal of ConnectionStrings in FinBuckle Finbuckle/Finbuckle.MultiTenant@f4e20db

Да.

  1. Это не происходит даже в ветке v2, поскольку FinBuckle удалил метод получения свойств MultiTenantContext.

https://github.com/fullstackhero/dotnet-webapi-starter-kit/blob/2af2c4f6d8947aa63153d992fba48740ffda1d30/api/framework/Infrastructure/Tenant/Services/TenantService.cs#L64C1-L68C15

изображение

  1. Удаление ConnectionStrings в FinBuckle Finbuckle/Finbuckle.MultiTenant@ f4e20db
    var setter = scope.ServiceProvider.GetRequiredService<IMultiTenantContextSetter>();
    setter.MultiTenantContext = new MultiTenantContext<FSHTenantInfo>
    {
        TenantInfo = tenant
    };

Yes, I am currently migrating it to 7.0.

I could see that using the below solves it.

 scope.ServiceProvider.GetRequiredService<IMultiTenantContextSetter>()
 .MultiTenantContext = new MultiTenantContext<FshTenantInfo>()
{
    TenantInfo = tenant
};