Skip to content

Fix UI test feedback - #9402

Open
Aries-0331 wants to merge 6 commits into
masterfrom
fix/ui-feedback
Open

Fix UI test feedback#9402
Aries-0331 wants to merge 6 commits into
masterfrom
fix/ui-feedback

Conversation

@Aries-0331

Copy link
Copy Markdown
Contributor

No description provided.

@seafile-dev seafile-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: Request changes

Blocking:

  • 依赖清单与锁文件版本不一致,npm ci 无法安装依赖;见行内评论。

Warnings:

  • 未发现其他可由当前差异明确复现的功能性问题。

Comment thread frontend/package.json

@seafile-dev seafile-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: Comment

Blocking:

  • 先前的依赖锁文件不一致已在当前提交中修复:package.jsonpackage-lock.json 均解析为 @seafile/seafile-calendar@1.0.14,干净执行 npm ci --ignore-scripts 成功。

Warnings:

  • 新增了一个未使用的 DirView 导入;见行内评论。
    Suggested fix: 删除该导入,保持变更通过项目的 lint 检查。

Comment thread frontend/src/metadata/components/popover/sort-popover/index.js Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants