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.
https://fullstackhero.net/dotnet-webapi-boilerplate/
MIT License
5.21k stars 1.56k forks source link

Is there any plan to migrate to Finbuckle 7.0.0 ? #925

Closed bvgnet closed 5 months ago

malisancube commented 6 months ago

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 https://github.com/Finbuckle/Finbuckle.MultiTenant/commit/f4e20db35fe9e31e1cfb37a667b1ba4b64ce6f3f
bvgnet commented 6 months ago

Да.

  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
    };
iammukeshm commented 6 months ago

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
};