[utils] Widen `isSafeDynamicKey` parameter type from `string` to `PropertyKey`
维护者通常 1 天内回复
还没有人认领这个 Issue。
- #9774 来自 @ZayanKhan-12 —— 已关闭，未合并
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 72/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- typescript
- 领域
- tooling
调研方向
首先在 MetaMask core monorepo 中定位 isSafeDynamicKey 及其现有测试。将参数类型从 string 更新为 PropertyKey，并验证相关测试和 TypeScript 检查能够通过 string、number 和 symbol 类型的键。
由索引模型根据 Issue 内容生成。
描述
isSafeDynamicKey should be able to handle all PropertyKey types.
- 主要语言
- TypeScript
- 星标
- 414
- 派生
- 310
- 平均合并
- 1 天 16 小时
- 30 天内合并 PR
- 415
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
MetaMask/core 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 1 天内回复
-
platform-api-docs: `scan` strategy misses ESM-only packages, which ship `.d.ts` rather than `.d.cts`未关闭
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
[utils] Replace the `uuid` dependency with `crypto.randomUUID()`可能已有人在做 @avantikonde 于 12 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 82/100
MetaMask/core#10210 · 1 个 reaction ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 1 天内回复
-
eth-json-rpc-middleware: `TransactionParamsStruct` rejects numeric `chainId`, breaking dapp transactions可能已有人在做 @truehazker 于 55 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 86/100
维护者通常 1 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 77/100
melgarafael/DeskcommCRM#3104 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
bug service: webapp
难度 2/5 1-3 小时 新手友好度 82/100
Ocelot-Social-Community/Ocelot-Social#10684 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
nicobailon/pi-subagents#2978 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 66/100
getsentry/sentry-conventions#732 · 1 条评论 ·
维护者通常 1 天内回复