get_store should have an optional store_id parameter
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 85
- Fork
- 36
- Merge trung bình
- 3 giờ 8 phút
- Pull request đã merge (30 ngày)
- 3
Mô tả
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 https://github.com/openfga/sdk-generator/issues/118
We should do something similar to what @jimmyjames mentions here:
https://github.com/openfga/java-sdk/issues/150#issuecomment-2666996341
- for requests that use the storeId in the path, update the request options object to include a
storeId- In
OpenFgaClient, if the options are not null andstoreIdis non-null and non-empty, use that to build the path. Otherwise, use thestoreIdconfigured on the client- Update the tests to verify that the
storeIdcan be overridden per request, and that the value on the client is used if not overridden.
In general, it should be pretty similar to how the
authorizationModelIdis allowed to be overridden per request.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với OpenFgaClient.get_store và các tùy chọn request được các phương thức của client sử dụng. So sánh hành vi override authorizationModelId hiện có, sau đó kiểm tra các test liên quan để xác minh rằng storeId cho từng request sẽ ghi đè store đã cấu hình và giá trị đã cấu hình vẫn là fallback.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- api
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100