Repository navigation
change field names - #62
Closed
Michael18811380328 wants to merge 1 commit into
Closed
Michael18811380328 wants to merge 1 commit into
Michael18811380328 wants to merge 1 commit into
Conversation
seafile-dev
reviewed
Oct 28, 2026
seafile-dev
left a comment
There was a problem hiding this comment.
结论:未发现本次文档改动引入的阻塞问题。发布前复核发现本 PR 已关闭且未合并,因此本审查仅作为补充记录,不表示字段迁移已经上线。
已核对当前文档与配套模板 PR #4,ProductName / UnitPrice 字段在 README、classic 和 ESM 中一致。历史 PR #61 已合并且为当前提交的祖先,其新增外链和截图仍完整保留:匿名访问可实际查看示例 base 的 Products、OrderItems、Orders 与产品记录;固定当前 SHA 的图片已逐张检查,覆盖配置、上传 ZIP、填写表单、提交成功和入库记录,图片路径有效。
当前提交通过 python tests/validate_docs.py --strict(0 errors / 0 warnings)及 mkdocs build --strict。本地按仓库默认未启用 git-committers 插件,因此不代表生产部署构建验证。配套模板当前 head 的两种模式打包及隔离提交契约验证通过,未向共享云端示例写入订单,未执行真实服务端到端测试。
待完成的发布步骤:示例 base 当前仍显示 Product_name / Unit_price。按任务约定,合并后需同步改成 ProductName / UnitPrice,并协调模板 PR #4 的启用,避免新模板读取旧 schema 时提示 Missing required fields。在该人工步骤完成前,不能视为字段迁移的完整上线验收;本次未修改云端表格。
未提供 SeaTable 设计文档。没有实质代码问题,不添加形式性行内评论。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.