Skip to content

feat(redenvelope): add risk control for red envelope claim and add QR Code Sharing - #258

Open
small-lovely-cat wants to merge 5 commits into
linux-do:masterfrom
small-lovely-cat:feat-red-envelope
Open

small-lovely-cat wants to merge 5 commits into
linux-do:masterfrom
small-lovely-cat:feat-red-envelope

Conversation

@small-lovely-cat

@small-lovely-cat small-lovely-cat commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

例行检查

  • 我已阅读并理解 贡献者公约
  • 我已阅读并同意 贡献者许可协议 (CLA),确认我的贡献将根据项目的 Apache2.0 许可证进行许可
  • 我知晓如果此 PR 并不做出实质性更改,或可被认为是为了PR被合并而提交PR的,则可能不会被合并

变更内容

  1. 为红包的claim接口添加风控逻辑,批量请求无效的红包ID将禁止一段时间内的请求。

  2. 发红包前端优化,添加二维码功能,支持面对面扫码领取。

image image

变更原因

功能优化 w~

@small-lovely-cat small-lovely-cat changed the title feat(redenvelope): add risk control for red envelope claim and add QR… feat(redenvelope): add risk control for red envelope claim and add QR Code Sharing Oct 4, 2026
Comment thread internal/apps/redenvelope/routers.go Outdated
Comment thread internal/db/redis.go
Comment thread internal/apps/redenvelope/risk.go Outdated
Comment thread internal/apps/redenvelope/risk.go
Comment thread frontend/components/common/trade/red-envelope.tsx Outdated
Comment thread frontend/components/common/trade/red-envelope.tsx
Comment thread internal/model/system_configs.go

@Aether-254 Aether-254 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

测试文件没写
还有一些小问题

Comment thread internal/apps/redenvelope/routers.go Outdated

@Aether-254 Aether-254 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

没啥问题了
之前的comment不知道为什么一直不能resolve

Failed to resolve thread

@small-lovely-cat

Copy link
Copy Markdown
Contributor Author

没啥问题了 之前的comment不知道为什么一直不能resolve

Failed to resolve thread

没有write access点不了的,已经resolve了 w)

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