Repository navigation
支持多级文件夹、整体移动与目录树搜索 - #1
Merged
Merged
Conversation
Owner
Author
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
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.
解决的问题
原来的单层文件夹无法表达“家庭 → 账单 → 年份”的归档结构,也无法重命名或整体移动目录。这次增加最多 8 层的文件夹层级,支持带子目录移动、面包屑导航和按目录树搜索。
改动与影响
验证证据
在 Windows、Python 3.14.6、独立的
qa-output/nested-folders数据目录完成:manage.py check:通过。manage.py makemigrations --check --dry-run:无未记录模型变化。FOLDERDOCS_TEST_OCR=1下manage.py test --verbosity 2:23 项全部通过,包含旧结构迁移、跨用户权限、POST/CSRF、循环/深度校验、目录移动/移除的数据保留,以及真实 Tesseract 对图片和扫描 PDF 的识别;失败重试路径另外使用 mock 验证。node qa/browser.cjs:Edge 无头浏览器验证登录、文件夹创建、上传、后台处理、搜索、下载、回收站恢复/永久删除和响应式布局。node qa/folders.cjs:Edge 无头浏览器验证三级目录创建、面包屑、递归搜索、重命名和带子目录移动、移动前后下载内容逐字节一致、末级目录安全移除和手机导航。git diff --check:通过。GitHub PR CI 在 Windows/Ubuntu、Python 3.12/3.14 的四组检查均通过(对应提交
238e6ee78a229c44bc77df72ca3bd4d0c73a9a63):运行结果。CI 默认跳过需本机 Tesseract 的集成测试,真实 OCR 已在本机单独验证。剩余风险
升级前应停止服务并备份整个数据目录。不同上级下使用同名目录后,不能直接回滚到旧的全局文件夹名称约束。本轮没有改动生产用户数据,也没有变更启动器或 worker 实现;未验证其他数据库部署或大规模目录性能。