Community Feedback Requested: Should we have named clients default to the default client if no default is registered?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 20/100
Hướng nghiên cứu
Bắt đầu bằng cách đọc DefaultVRChatClientFactory.cs và VRChatServiceCollectionExtensions.cs tại các vị trí được liên kết để hiểu cách CreateClient() chọn một client có tên khi chưa đăng ký client mặc định nào. Xem xét các phương án thay thế được đề xuất trong issue và thu thập phản hồi cộng đồng được yêu cầu; công việc được xem là hoàn tất khi hành vi dự kiến đã được quyết định và phạm vi triển khai đã được ghi lại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
When using DI, if IVRChatClientFactory.CreateClient() is called, and no default IVRChat was registered, then it will return the first available named client.
This may lead to unexpected behavior or bugs when consuming the library (and unaware of what the actual problem is), but also may make life easier when clients are left unregistered.
I wanted to obtain community feedback before implementing a complete change.
The implementation of this behavior is located here and here
- Ngôn ngữ chính
- C#
- Star
- 6
- Fork
- 2
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
Issue khác của binn/VRChat.API.Client
-
TODO: Implement unit tests Đang mởenhancement help wanted
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
-
TODO: Document all behavior. Đang mởdocumentation good first issue help wanted
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
TODO: Document all use-cases Đang mởdocumentation good first issue
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
-
TODO: Update XmlDoc Đang mởdocumentation good first issue
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
Tất cả issue của binn/VRChat.API.Client
Issue tương tự
-
effort:S P3 refactor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
nightscout/nocturne#1532 ·
-
core dependencies
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 80/100
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
C#/.NET Roslyn LSP Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Suspicious code fragments Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
DotNetNext/SqlSugar#1458 ·