-
Notifications
You must be signed in to change notification settings - Fork 15
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
get_store should have an optional store_id parameter #9
Comments
Same issue with delete_store. In the current implemantation, we have to create a specific client_api for every store that we would like to delete. |
Hi @misterflop Currently, our SDKs now expect you to generate a new client per store. In the future (as part of openfga/sdk-generator#118) we are exploring offering two different interfaces which would allow us to have one that allows calling one with dynamic storeIds, but I don't expect that to be soon Can you elaborate on your use-case where you would need to call multiple stores with the same client? |
Hello, Thank you for your answer. I see two use cases on my side:
The openFGA API allows to request per store ID. The SDK should ideally have the same contract. |
In my organization, we are also facing the similar kind of Issue. To solve that we have built another wrapper microservice around the openfga sdk. You can also try the same. It will be more flexible for you |
interested @rhamzeh Please assign |
@evansims can you please assign this to me? |
👋🏻 Thanks for the offer @gagandeepp. Sorry about this, we feel this issue isn't well defined for hacktoberfest as it requires working on some pieces of the SDK generation that are quite awkward to update. We've specifically labelled issues that we're looking for help with with the |
In the current implementation, get_store return the OpenFGA store specified in the client_api configuration.
Except by listing all the stores with list_stores, I don't see any other way to get a specific store different of the one defined in the configuration.
A way to arrange that would be to add an optional store_id parameter to get_store.
Part of openfga/sdk-generator#118
The text was updated successfully, but these errors were encountered: