Skip to content

feat: ModifyFitnessScore#37

Closed
Jiahe114514 wants to merge 1 commit intomainfrom
feat/ModifyFitnessScore
Closed

feat: ModifyFitnessScore#37
Jiahe114514 wants to merge 1 commit intomainfrom
feat/ModifyFitnessScore

Conversation

@Jiahe114514
Copy link
Contributor

@Jiahe114514 Jiahe114514 commented Feb 28, 2026

Summary by CodeRabbit

发布说明

  • 新功能
    • 新增体质评分修改接口,允许更新已保存的体质测试数据,支持修改身高、体重、BMI、50米跑、站立长跳、坐位体前屈、800米跑、1000米跑、仰卧起坐、引体向上、肺活量等指标及总分。

@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d7a767e and 872b17a.

📒 Files selected for processing (1)
  • campusapis/staff/v1/campus.proto

整体概述

campusapis/staff/v1/campus.proto 中新增了 ModifyFitnessScore RPC 方法及其对应的 ModifyFitnessScoreRequest 消息,用于支持健身成绩的修改操作。同时从 DeleteFitnessScoreRequest 移除了 OpenAPI schema 注解。

变更内容

内聚组 / 文件 摘要
Proto API 定义
campusapis/staff/v1/campus.proto
添加 ModifyFitnessScore RPC 方法(支持 PUT HTTP 绑定),新增 ModifyFitnessScoreRequest 消息类型(包含 17 个字段用于完整的健身成绩数据),移除 DeleteFitnessScoreRequest 的 OpenAPI schema 注解。

预计代码审查工作量

🎯 2 (简单) | ⏱️ ~10 分钟

可能相关的 PR

  • chore: fitness data storage #32:同样修改 campusapis/staff/v1/campus.proto 中的健身成绩相关 API 接口——在本 PR 中新增了修改健身成绩的 RPC 和对应消息定义。

兔兔之诗

🐰 健身成绩需更新,
新 RPC 来助力,
十七个字段齐整齐,
修改操作妥妥的!✨
文件改动虽轻巧,
功能完善不可少。

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/ModifyFitnessScore

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

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