Skip to content

fix(consumer-web): 修复评分表单字段绑定、时长校验与剧透状态同步问题 - #1

Closed
SXP-Simon wants to merge 1 commit into
Hikarinagi:mainfrom
SXP-Simon:fix/rate
Closed

fix(consumer-web): 修复评分表单字段绑定、时长校验与剧透状态同步问题#1
SXP-Simon wants to merge 1 commit into
Hikarinagi:mainfrom
SXP-Simon:fix/rate

Conversation

@SXP-Simon

Copy link
Copy Markdown
Contributor
  1. 移除 FormItem 中 Checkbox 与 InputNumber 冗余的手动事件绑定,修复剧透勾选与状态冲突问题
  2. 重构 rate.schema.ts 中的 time_to_finish_hours 与 dimension 规则,支持退格清空(防 NaN)、小数时长换算与显式超限拦截
  3. 移除 InputNumber 静默截断属性,统一由 Valibot Schema 进行显式校验与错误提示
  4. 新增 12 个 Vitest 自动化单元测试用例,覆盖 4 类媒体的边界与换算场景

1. 移除 FormItem 中 Checkbox 与 InputNumber 冗余的手动事件绑定,修复剧透勾选与状态冲突问题
2. 重构 rate.schema.ts 中的 time_to_finish_hours 与 dimension 规则,支持退格清空(防 NaN)、小数时长换算与显式超限拦截
3. 移除 InputNumber 静默截断属性,统一由 Valibot Schema 进行显式校验与错误提示
4. 新增 12 个 Vitest 自动化单元测试用例,覆盖 4 类媒体的边界与换算场景
@Ringyuki
Ringyuki self-requested a review August 22, 2026 13:09
@Ringyuki Ringyuki added the bug Something isn't working label Aug 22, 2026

@Ringyuki Ringyuki left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@Ringyuki

Copy link
Copy Markdown
Member

Accepted for upstream integration. The imported private pull request is now running its upstream validation and will be mirrored back here after review and merge.

@Ringyuki

Copy link
Copy Markdown
Member

Merged into the upstream monorepo and published by the public source mirror. Thank you for the contribution!

@Ringyuki Ringyuki closed this Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants