microsoft / microsoft/FeatureManagement-JavaScript
Missing await causes true evaluation
Aberta
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- TypeScript
- Estrelas
- 19
- Forks
- 6
- Merge médio
- 2d 3h
- PRs com merge (30d)
- 4
Descrição
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.
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece localizando a implementação de isEnabled e qualquer uso ou teste relacionado e, em seguida, reproduza a condicional mostrada na issue. A issue não fornece nenhum comportamento-alvo além de identificar o problema de missing-await, portanto a resolução pretendida e os critérios de conclusão precisam ser esclarecidos antes da implementação.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- javascript, typescript
- Domínio
- api
- Tipo de issue
- Bug
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Status de atividade
- Estagnada
- Clareza
- Precisa de esclarecimento
- Facilidade para iniciantes
- 25/100