Skip to content

Comments

feat(region): make qqGroups label optional and update RegionSelector#32

Merged
PaiJi merged 1 commit intomainfrom
feat/allow-field-nullable
Feb 3, 2026
Merged

feat(region): make qqGroups label optional and update RegionSelector#32
PaiJi merged 1 commit intomainfrom
feat/allow-field-nullable

Conversation

@PaiJi
Copy link
Member

@PaiJi PaiJi commented Feb 3, 2026

Summary by CodeRabbit

Bug Fixes

  • 事件API中标签字段现已可选,提高表单灵活性

Refactor

  • 调整组织创建API请求体结构
  • 优化区域编辑器占位符处理方式

@PaiJi PaiJi merged commit cc3377e into main Feb 3, 2026
2 checks passed
@PaiJi PaiJi deleted the feat/allow-field-nullable branch February 3, 2026 06:30
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

该PR对三个文件进行了小规模调整:放宽了事件API中标签字段的验证要求,修改了组织API的请求体封装方式,以及优化了地区编辑组件的属性传递机制。

Changes

聚合 / 文件 摘要
API验证调整
src/api/dashboard/event.ts
EditEventApiBody中qqGroups对象内的label字段从必需string改为可选string,允许该字段可被省略。
API请求体结构改动
src/api/dashboard/organization.ts
createOrganization API调用修改为直接发送请求体,而非将其包装在带"organization"属性的对象中,简化了POST请求的数据结构。
UI属性传递优化
src/pages/dashboard/region/edit.tsx
RegionEditorContent组件改为通过antdSelectProps对象传递placeholder属性,而非直接传递,改善了props的转发机制。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 一行一改,三处调和,
API轻装上阵,UI属性更妥,
验证放宽,结构简化,
小兔欢快地跳过这些变化!✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/allow-field-nullable

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

🚀 预览部署完成!

📱 预览地址: https://fcc-dashboard-0rak2krto4.edgeone.run?eo_token=b066f89f56d4e516141816a1f1651690&eo_time=1770100325

此预览链接将在部署完成后可用,请稍等片刻再访问。


由 GitHub Actions 自动生成

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.

1 participant