Skip to content

fix: #34 功能需求 + UserProfile 扩展信息#69

Open
dajiaohuang wants to merge 5 commits into
zhtyyx:mainfrom
dajiaohuang:main
Open

fix: #34 功能需求 + UserProfile 扩展信息#69
dajiaohuang wants to merge 5 commits into
zhtyyx:mainfrom
dajiaohuang:main

Conversation

@dajiaohuang

Copy link
Copy Markdown

修复内容

Issue #34 — 功能需求反馈

1. 库存入库/出库/调整按钮无响应

  • 修复 inventory_list.html 中入库/出库/调整按钮 href="#" → 正确链接
  • 按钮点击后自动预填选中商品

2. 商品批量导入导出 + 颜色/尺码自定义

  • 新增商品 CSV 导入/导出功能(含模板和格式说明)
  • 修复 product_service 中字段名不匹配(retail_priceprice
  • 新增颜色/尺码自定义管理 CRUD(列表、新建、编辑、删除)
  • 分类管理页新增颜色/尺码管理入口

3. 缺少退货功能

  • Sale 模型新增 RETURNED(已退货)状态
  • 新增 sale_return 视图:恢复库存 + 退还余额 + 扣回积分 + 操作日志
  • 销售单详情页:已完成订单显示退货按钮,状态栏区分四种状态

Issue #23 — UserProfile 扩展信息

  • 新增 UserProfile 模型:手机号、部门、工号、备注
  • post_save signal 自动为新用户创建空 profile
  • 迁移中补建已有用户的 profile
  • 管理员创建/编辑用户页支持扩展字段
  • 用户列表新增手机号/部门列
  • 用户详情页展示扩展信息
  • 新增 /profile/ 页面:用户可自行编辑个人信息
  • 不影响原有登录流程

文件变更

  • 30 files, +914 / −40 lines

🤖 Generated with Claude Code

dajiaohuang and others added 5 commits June 20, 2026 00:08
fix: address issues zhtyyx#34 and zhtyyx#23

Issue zhtyyx#34 - 功能需求反馈:
- 修复库存入库/出库/调整按钮无响应问题(href="#" 改为正确 URL + 预填商品)
- 新增商品批量导入(CSV)和导出功能(修复字段名不匹配问题)
- 新增颜色/尺码自定义管理(CRUD + 模板)
- 新增销售退货功能(恢复库存、退还余额、标记状态)
- 为 Sale 模型添加 RETURNED(已退货)状态

Issue zhtyyx#23 - 用户表缺少扩展信息:
- 新增 UserProfile 模型(OneToOne → User),含手机号、部门、工号、备注
- 用户创建/编辑/详情/列表页均支持扩展信息字段

Co-Authored-By: Claude <noreply@anthropic.com>
@
fix: user profile 改为可选,模板安全访问避免崩溃

- user_list 视图用 try/except 安全获取 profile
- user_detail / user_update 视图传递 profile 到模板 context
- 所有模板改用安全变量替代直接访问 user.profile
- profile 不存在时正常显示 "-"

Co-Authored-By: Claude <noreply@anthropic.com>
@
revert: 移除 UserProfile 相关代码,仅保留 Sale RETURNED 状态

- 撤回 UserProfile 模型、视图、模板的改动
- 重新生成迁移,仅包含 Sale status 新增 RETURNED
- 其余 issue zhtyyx#34 修复保持不变

Co-Authored-By: Claude <noreply@anthropic.com>
@
feat: UserProfile 扩展信息(注册自动创建,用户/管理员均可编辑)

- UserProfile 模型:phone/department/employee_id/notes
- post_save signal 自动为每个新 User 创建空 profile
- 数据迁移:为已有用户补充 profile
- 管理员:user_create/user_update 可编辑扩展字段
- 用户:/profile/ 页面可自行编辑个人信息
- 列表新增手机号/部门列,详情页展示扩展信息
- 不影响原有登录/注册流程

Co-Authored-By: Claude <noreply@anthropic.com>
@
fix: 恢复 user_list 中被误删的邮箱列

Co-Authored-By: Claude <noreply@anthropic.com>
@
@zhtyyx

zhtyyx commented Jun 21, 2026

Copy link
Copy Markdown
Owner

@dajiaohuang 方便加个联系方式聊一下吗, zhtyyx@gmail.com

@zhtyyx

zhtyyx commented Jun 21, 2026

Copy link
Copy Markdown
Owner

感谢提交 PR,这里面确实有一些有价值的修复,比如库存入库/出库/调整按钮、商品导入导出、退货、用户扩展信息等方向都能对应现有 issue。
但这个 PR 目前不适合直接合并,主要原因是改动范围太大,把多个互不相关的功能和修复放在了一起:Issue #34、Issue #23、退货流程、UserProfile、颜色/尺码管理、商品导入导出等都在同一个 PR 里,后续 review、回滚和定位问题都会比较困难。

另外其中有几处高风险点还需要进一步处理:

  1. 退货会修改库存、会员余额、积分、消费次数和销售单状态,属于财务/库存核心路径,目前缺少对应测试,也缺少更明确的权限控制,不能只依赖登录态。
  2. 商品导入页里的示例 CSV 字段和实际校验字段不一致,用户按示例导入可能会失败。
  3. 新增的颜色/尺码管理、商品导入导出、退货等写操作需要补齐权限边界,避免普通登录用户也能操作敏感功能。
  4. 这个 PR 没有 CI 或测试结果支撑,改动文件较多,直接合并风险较高。

建议把这个 PR 拆成几个更小的 PR:

  • 一个 PR 只修库存按钮无响应;
  • 一个 PR 做商品导入导出,并修正 CSV 示例和测试;
  • 一个 PR 做颜色/尺码管理;
  • 一个 PR 单独做退货流程,补充库存、余额、积分相关测试;
  • UserProfile 扩展信息单独讨论和提交。

这样每个 PR 都能独立 review、测试和合并,也更容易保证主分支稳定。

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