Skip to content

feat(pay): 补齐商家转账收款样式参数 - #4132

Merged
binarywang merged 2 commits into
binarywang:developfrom
CalmChih:develop
Oct 10, 2026
Merged

binarywang merged 2 commits into
binarywang:developfrom
CalmChih:develop

Conversation

@CalmChih

Copy link
Copy Markdown
Contributor

为 TransferBillsRequest 增加 user_recv_style.type 字段,支持设置收款确认页或红包样式。补充请求体序列化回归测试,验证显式设置和省略参数的行为。

为 TransferBillsRequest 增加 user_recv_style.type 字段,支持设置收款确认页或红包样式。补充请求体序列化回归测试,验证显式设置和省略参数的行为。

Copilot AI 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.

🟡 Changes recommended

需保留旧公共构造函数的兼容性,并修复设置收款样式后的 Java 序列化失败问题。

2 open findings
What changed in this PR

为微信支付商家转账请求补充收款样式参数,支持选择收款确认页或红包样式。

Changes:

  • 新增 user_recv_style.type 字段及说明。
  • 补充显式设置与省略参数时的请求序列化回归测试。
File Description
weixin-java-pay/​src/​test/​java/​com/​github/​binarywang/​wxpay/​service/​impl/​TransferUserAuthorizationApiCompatibilityTest.java 验证收款样式字段的发送与省略行为。
weixin-java-pay/​src/​main/​java/​com/​github/​binarywang/​wxpay/​bean/​transfer/​TransferBillsRequest.java 新增收款样式字段和嵌套模型。

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Retain original constructor while adding default for userRecvStyle.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@binarywang

Copy link
Copy Markdown
Owner

麻烦处理下review 建议,谢谢

@CalmChih

CalmChih commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

麻烦处理下review 建议,谢谢

你好,序列化使用的是Gson工具包,不会有这个问题吧,需要修复这个吗?

@binarywang

Copy link
Copy Markdown
Owner

麻烦处理下review 建议,谢谢

你好,序列化使用的是Gson工具包,不会有这个问题吧,需要修复这个吗?

是的,当前接口使用 Gson,新增类型未实现 Serializable 不影响 JSON 请求序列化。这里的 review 针对的是外层请求对象已实现 Serializable 后,下游使用 Java 原生序列化的场景。建议新增类型也实现 Serializable 并声明 serialVersionUID,以保持模型一致性。
不过如果确实觉得没必要,直接resolve这条review吧

@CalmChih

Copy link
Copy Markdown
Contributor Author

麻烦处理下review 建议,谢谢

你好,序列化使用的是Gson工具包,不会有这个问题吧,需要修复这个吗?

是的,当前接口使用 Gson,新增类型未实现 Serializable 不影响 JSON 请求序列化。这里的 review 针对的是外层请求对象已实现 Serializable 后,下游使用 Java 原生序列化的场景。建议新增类型也实现 Serializable 并声明 serialVersionUID,以保持模型一致性。 不过如果确实觉得没必要,直接resolve这条review吧

TransferBillsRequest内的另外两个类也没实现Serializable,我觉得还是暂时保持一致,resolve了。

@binarywang
binarywang merged commit f795960 into binarywang:develop Oct 10, 2026
1 check passed
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.

3 participants