Community Feedback Requested: Should we have named clients default to the default client if no default is registered?

Offen

#4 erstellt am 26.11.2021 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 20/100

Issue-Typ: Feature
Klarheit: Muss geklärt werden
Aktivitätsstatus: Veraltet
Tech-Stack: csharp
Bereich: backend

Rechercherichtung

Beginne damit, DefaultVRChatClientFactory.cs und VRChatServiceCollectionExtensions.cs an den verlinkten Stellen zu lesen, um zu verstehen, wie CreateClient() einen benannten Client auswählt, wenn kein Standard registriert ist. Prüfe die im Issue vorgeschlagenen Alternativen und hole das angeforderte Community-Feedback ein; abgeschlossen ist die Aufgabe, wenn das erwartete Verhalten festgelegt und der Umfang der Implementierung dokumentiert ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug enhancement help wanted question

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

Vorherrschende Sprache: C#
Sterne: 6
Forks: 2
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus binn/VRChat.API.Client

TODO: Implement unit tests Offen

enhancement help wanted

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100

binn/VRChat.API.Client#5 · Erstellt am 26.11.2021
TODO: Document all behavior. Offen

documentation good first issue help wanted

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

binn/VRChat.API.Client#3 · Erstellt am 26.11.2021
TODO: Document all use-cases Offen

documentation good first issue

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

binn/VRChat.API.Client#2 · Erstellt am 26.11.2021
TODO: Update XmlDoc Offen

documentation good first issue

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100

binn/VRChat.API.Client#1 · Erstellt am 26.11.2021

Alle Issues in binn/VRChat.API.Client