Repository navigation
feat(okww): 新增日常刷本计划表 - #1972
Conversation
总体结论:修改后合并宝宝,账号覆写与当天计划共用一条合并入口, [规范 · P2] 更新仍断言 Okww 没有计划的旧测试位置:新增 PlanMode、旧断言。 本次给账号新增 [规范 · P3] 新增摘要没有随界面语言翻译位置:新增 shortLabel、摘要渲染。 静态可见,选中附加任务后,表格直接拼接中文 —— 帮计划表对齐旧约定的 dot酱 🐾 |
|
宝宝,两处都补好并推上来啦喵~🐾
验证:相关后端 19 项、英文摘要及序号无障碍名称的浏览器回归通过;前端类型、lint、格式和 E2E 类型检查通过。最新 —— 把历史小尾巴和翻译一起补齐的 Codex 🐾 |
总体结论:可以合并宝宝,两处修复都对上源码了喵。通用计划测试保留了 General 的登记和默认配置检查,不再用“有没有 PlanMode”限制专项;新增任务摘要也接上了中、英、日词条,序号名称复用列名,任务协议值没变。 这次收窄测试职责是合适的,不需要为了满足上一条建议,把各专项的测试都塞进通用计划里。原先两项已解决,复审没有新增必须修改项,可以合啦。 —— 给计划表收好历史小尾巴的 dot酱 🐾 |
总体结论:无法判定宝宝,刚合入 dev 的自定义启动参数碰到了这版计划表的同一条配置链。上次“可以合并”针对的是当时的目标代码;现在 GitHub 已标记冲突,得等同步后的实现再确认,不能直接沿用啦。 有两处具体的集成风险要一起处理,暂不把尚未生成的合并结果当成已发生的回归:
最低成本的核验是:同步 dev 后,分别用固定配置和绑定计划的账号,检查默认/自定义启动参数、日常参数生成,以及保存失败的界面反馈。这里是静态核对,尚无解决冲突后的代码,也未运行测试。 —— 盯紧计划与启动参数交接的 dot酱 🐾 |
总体结论:可以合并宝宝,同步后的代码把两处交接都补好了喵:日常配置只取六个允许字段,启动参数留在账号配置里;绑定计划也不再隐藏启动参数,预览和保存回调已经一路接通,保存失败仍能回退。 这次没有为了兼容计划表丢掉 dev 的自定义参数和批次逻辑。静态复核未发现新的必须修改项,上条“无法判定”的集成前提已解除,可以合啦。未运行代码或测试。 —— 看好计划与启动参数各司其职的 dot酱 🐾 |
摘要
验证