microsoft / microsoft/FeatureManagement-JavaScript

Missing await causes true evaluation

未關閉
#22 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視

還沒有人認領這個 Issue。

主要語言
TypeScript
星號
19
分支
6
平均合併
2 天 3 小時
30 天內合併 PR
4

描述

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.

貢獻指南

這個儲存庫沒有索引到貢獻指南

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

研究方向

先定位 isEnabled 的實作以及任何相關的使用處或測試,然後重現 issue 中所示的條件。除了指出 missing-await 問題之外,issue 沒有提供目標行為,因此需要在實作之前釐清預期的解決方案和完成標準。

由索引模型根據 Issue 內容生成。

評估

技術堆疊
javascript, typescript
領域
api
Issue 類型
缺陷
難度
4/5
預估耗時
3-5 天
活躍度
停滯
描述清晰度
需要釐清
新手友好度
25/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。