feat: 支持活动报名与签到接口 - #986
Conversation
新增活动详情中的个人参与状态、报名与取消报名接口,并统一活动 API 错误结构。完善签到校验与活动业务函数复用,补充并发、权限、容量和状态转换测试。
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6b87c16117
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
整体没啥问题,但是还有几个小问题可能需要你同时修改一下
控制“活动开始前一小时才生成/显示签到码”的核心代码在: 关键判断是第 334 行: return now >= activity.start - timedelta(hours=1)调用链如下:
另外,“扫码后是否允许签到”还有两处独立的一小时判断,这两个是合理的。 我的理解是码可以提前任意时间生成,但是签到不允许提前一小时签到。 |
允许签到码提前生成,同时保留开始前一小时的实际签到限制。将报名账户状态和签到活动状态的校验移入事务锁内。
已按意见修改:签到码现在可以提前生成,实际签到仍限制在活动开始前一小时内;同时调整了报名和签到的并发校验。相关测试均已通过,麻烦再帮忙审核一下,谢谢! |
背景
需求池中的“新生之旅报名与签到”需要让新生之旅系列讲座和太舞徒步分别支持报名、取消报名、现场扫码签到,并保留参与记录。
本 PR 实现通用活动能力;正式活动的时间、地点、人数和介绍继续由后台活动数据配置,不在代码中写死。
主要修改
影响
验证
git diff --check通过。