Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
bvgnet opened this issue Apr 23, 2024 · 3 comments
Open

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

bvgnet opened this issue Apr 23, 2024 · 3 comments

Comments

@bvgnet
Copy link

bvgnet commented Apr 23, 2024

No description provided.

@malisancube
Copy link

malisancube commented Apr 26, 2024

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

@bvgnet
Copy link
Author

bvgnet commented Apr 27, 2024

Да.

  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
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants