Finbuckle.MultiTenant icon indicating copy to clipboard operation
Finbuckle.MultiTenant copied to clipboard

Add optional support for GetAllAsync() via HttpRemoteStore

Open Blackburn29 opened this issue 1 year ago • 2 comments

I ran into this issue while converting from an EF store to HTTP in a project. My comment on issue https://github.com/Finbuckle/Finbuckle.MultiTenant/issues/808 mainly provides the need for this and I managed to implement it manually within my app, however I'd like to mainline this for future projects. I can definitely understand why people would not want this, which is why I based the implementation off the existing route template and will throw NotImplementedException in the case of the external http service having an undefined route for this implementation.

Feedback is greatly appreciated!

Blackburn29 avatar Jul 02 '24 19:07 Blackburn29

@dotnet-policy-service agree

Blackburn29 avatar Jul 02 '24 20:07 Blackburn29

thanks @Blackburn29

AndrewTriesToCode avatar Jul 05 '24 20:07 AndrewTriesToCode