microsoft / microsoft/FeatureManagement-JavaScript
Missing await causes true evaluation
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 19
- Forks
- 6
- Ø Merge
- 2 T. 3 Std.
- Gemergte PRs (30 T.)
- 4
Beschreibung
if (temp.isEnabled("foo")) {
button.className = "cool";
button.textContent = "Cool Button";
} else {
button.className = "lame";
button.textContent = "Lame Button";
}
We will always run the "cool" path here, because I didn't put an await on the isEnabled call. I think this is bad.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Implementierung von isEnabled sowie jede zugehörige Verwendung oder Tests zu finden, und bilde anschließend die im Issue gezeigte Bedingung nach. Das Issue gibt über die Identifizierung des missing-await-Problems hinaus kein Zielverhalten vor, daher müssen die beabsichtigte Lösung und die Kriterien für den Abschluss vor der Implementierung geklärt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, typescript
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100