Tracker method calling part needs to be refactored
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 25/100
- Issue 类型
- 重构
- 描述清晰度
- 需要澄清
- 活跃度
- 停滞
- 技术栈
- rust
- 领域
- blockchain
调研方向
首先比较 types/src/transaction/shard.rs、types/src/transaction/action.rs 和 types/src/transaction/transaction.rs 中的 tracker() 实现,然后检查相关的调用方函数以及 PR #1548 的变更。跟踪现有的 Action-to-ShardTransaction 转换和 WrapCCC 路径。当 tracker API 及其调用方具有一致的语义且不存在冗余转换时,即视为完成。
由索引模型根据 Issue 内容生成。
描述
Now the tracker() method can be called through the structs ShardTransaction, Action and Transaction.
Some caller function code is now using useless conversion from Action to ShardTransaction though the conversion was already implemented inside of the tracker() of Action.
As well as the above, after the pr#1548, the implementations will become inconsistent. Previously the tracker was needed only for asset transactions, but now the WrapCCC needs tracker though it's not an asset transaction.
From this semantic inconsistency, there is an implementation gap between tracker() of Action and the others. Therefore, the refactoring needs for tracker() and related caller functions not to be confused in the future.
- 主要语言
- Rust
- 星标
- 256
- 派生
- 50
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
CodeChain-io/codechain 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 25/100
CodeChain-io/codechain#2002 ·
-
难度 4/5 3-5 天 新手友好度 35/100
CodeChain-io/codechain#1993 · 2 条评论 ·
-
难度 4/5 3-5 天 新手友好度 35/100
CodeChain-io/codechain#1987 · 1 条评论 ·
-
performance rpc
难度 4/5 3-5 天 新手友好度 35/100
CodeChain-io/codechain#1973 ·
-
难度 3/5 1-2 天 新手友好度 35/100
CodeChain-io/codechain#1971 ·
查看 CodeChain-io/codechain 的全部 Issue
相似的 Issue
-
bug
难度 1/5 1 小时以内 新手友好度 85/100
-
难度 2/5 1-3 小时 新手友好度 75/100
yantrikos/yantrik-os#255 ·
-
bug CLI custom-model
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 70/100
raphamorim/rio#1956 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
rust-bitcoin/rust-bitcoin#6930 · 1 条评论 ·