Missing await causes true evaluation

Ouverte
#22 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
25/100
Type d'issue
Bug
Clarté
À clarifier
Activité
À l'abandon
Stack technique
javascript, typescript
Domaine
api

Piste de recherche

Commencez par localiser l’implémentation de isEnabled ainsi que toute utilisation ou tout test associé, puis reproduisez la condition présentée dans l’issue. L’issue ne fournit aucun comportement cible au-delà de l’identification du problème de missing-await ; la résolution attendue et les critères d’achèvement doivent donc être clarifiés avant l’implémentation.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
TypeScript
Étoiles
19
Forks
6
Merge moyen
2 j 3 h
PR mergées (30 j)
4

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de microsoft/FeatureManagement-JavaScript

Toutes les issues de microsoft/FeatureManagement-JavaScript

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.