Community Feedback Requested: Should we have named clients default to the default client if no default is registered?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 20/100
Línea de trabajo
Comienza leyendo DefaultVRChatClientFactory.cs y VRChatServiceCollectionExtensions.cs en las ubicaciones enlazadas para entender cómo CreateClient() selecciona un cliente con nombre cuando no hay ningún cliente predeterminado registrado. Revisa las alternativas propuestas en el issue y recopila los comentarios solicitados de la comunidad; se considera terminado cuando se haya decidido el comportamiento esperado y se haya documentado el alcance de la implementación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- C#
- Estrellas
- 6
- Forks
- 2
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de binn/VRChat.API.Client
-
TODO: Implement unit tests Abiertoenhancement help wanted
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
-
TODO: Document all behavior. Abiertodocumentation good first issue help wanted
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
TODO: Document all use-cases Abiertodocumentation good first issue
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
TODO: Update XmlDoc Abiertodocumentation good first issue
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
Todos los issues de binn/VRChat.API.Client
Issues similares
-
effort:S P3 refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
nightscout/nocturne#1532 ·
-
core dependencies
Dificultad 1/5 Menos de una hora Aptitud para principiantes 80/100
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
C#/.NET Roslyn LSP Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Suspicious code fragments Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
DotNetNext/SqlSugar#1458 ·