Skip to content

支持简历子项排序与插入 - #89

Merged
LingyiChen-AI merged 1 commit into
LingyiChen-AI:mainfrom
Silas-Zhu:feat/item-sort
Aug 15, 2026
Merged

支持简历子项排序与插入#89
LingyiChen-AI merged 1 commit into
LingyiChen-AI:mainfrom
Silas-Zhu:feat/item-sort

Conversation

@Silas-Zhu

Copy link
Copy Markdown
Contributor

No description provided.

@vercel

vercel Bot commented Jul 2, 2026

Copy link
Copy Markdown

@Silas-Zhu is attempting to deploy a commit to the twwch's projects Team on Vercel.

A member of the Team first needs to authorize it.

@LingyiChen-AI
LingyiChen-AI merged commit 64ae510 into LingyiChen-AI:main Aug 15, 2026
1 check failed
@LingyiChen-AI

Copy link
Copy Markdown
Owner

已合并进 main(64ae510)。感谢贡献 👍

验证情况

不只看了 diff,用生产构建起了服务,在真实编辑器里逐项操作验证:

操作 结果
初始 公司A → 公司B → 公司C
第 2 条「在上方插入」 公司A → (空) → 公司B → 公司C
删除刚插入的 公司A → 公司B → 公司C
第 1 条「下移」 公司B → 公司A → 公司C
边界 首条「上移」、末条「下移」均正确禁用
刷新页面 公司B → 公司A → 公司C

最后一行是关键——顺序确实落库了,不只是前端状态。

类型检查通过,162 个测试全过。

顺带说明

moveItem 的实现(splice 出来再 splice 回目标位)带了边界检查,insertItem 插在正确的下标上,逻辑没问题。另外这个 PR 把好几处 onUpdate({ items: updated } as any)as any 去掉了,这是额外的收获。

合并时只有 .gitignore 一处冲突:你加的 .vscode 和 main 上新加的 release/ 落在同一位置,两个都保留了。

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