Community Feedback Requested: Should we have named clients default to the default client if no default is registered?
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 20/100
Piste de recherche
Commencez par lire DefaultVRChatClientFactory.cs et VRChatServiceCollectionExtensions.cs aux emplacements liés afin de comprendre comment CreateClient() sélectionne un client nommé lorsqu'aucun client par défaut n'est enregistré. Examinez les alternatives proposées dans l'issue et recueillez les retours de la communauté demandés ; le travail est considéré comme terminé lorsque le comportement attendu est décidé et que le périmètre de l'implémentation est documenté.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- C#
- Étoiles
- 6
- Forks
- 2
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de binn/VRChat.API.Client
-
TODO: Implement unit tests Ouverteenhancement help wanted
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
-
TODO: Document all behavior. Ouvertedocumentation good first issue help wanted
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
TODO: Document all use-cases Ouvertedocumentation good first issue
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
-
TODO: Update XmlDoc Ouvertedocumentation good first issue
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
Toutes les issues de binn/VRChat.API.Client
Issues similaires
-
effort:S P3 refactor
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
nightscout/nocturne#1532 ·
-
core dependencies
Difficulté 1/5 Moins d'une heure Accessibilité débutants 80/100
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
C#/.NET Roslyn LSP Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Suspicious code fragments Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
DotNetNext/SqlSugar#1458 ·