Skip to content

同步新字段 #3626

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

同步新字段 #3626

wants to merge 1 commit into from

Conversation

ixqbar
Copy link

@ixqbar ixqbar commented Jun 26, 2025

No description provided.

@binarywang binarywang requested a review from Copilot June 27, 2025 09:59
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces support for a new jump target option in card packages, updates background color options, and syncs new fields in the FavorStocksCreateRequest payload.

  • Added a JumpTargetEnum to represent card-package jump targets.
  • Extended BackgroundColorEnum with four new color constants.
  • Introduced jumpTarget, miniProgramAppid, and miniProgramPath fields in PatternInfo.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
weixin-java-pay/.../enums/JumpTargetEnum.java New enum for card-package jump targets
weixin-java-pay/.../enums/BackgroundColorEnum.java Added COLOR081, COLOR082, COLOR101, COLOR102 constants
weixin-java-pay/src/.../FavorStocksCreateRequest.java Added jumpTarget, miniProgramAppid, miniProgramPath fields to PatternInfo
Comments suppressed due to low confidence (2)

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/marketing/enums/JumpTargetEnum.java:32

  • The Javadoc for the value field says "批次类型" (batch type), which doesn’t match its purpose. It should explain that this field holds the jump target code value, e.g., "跳转目标编码".
   */

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/marketing/FavorStocksCreateRequest.java:400

  • [nitpick] Consider adding or updating unit tests to verify serialization/deserialization of the new jumpTarget, miniProgramAppid, and miniProgramPath fields in PatternInfo.
    @SerializedName("jump_target")

@binarywang
Copy link
Owner

麻烦准确描述下更新内容,而不是简单的“同步新字段”。
你要说明是针对哪个接口所做的。

@ixqbar
Copy link
Author

ixqbar commented Jun 29, 2025

https://pay.weixin.qq.com/doc/v3/merchant/4012534633

image

可选取值:

COLOR010: 色值 #63b359

COLOR020: 色值 #2c9f67

COLOR030: 色值 #509fc9

COLOR040: 色值 #5885cf

COLOR050: 色值 #9062c0

COLOR060: 色值 #d09a45

COLOR070: 色值 #e4b138

COLOR080: 色值 #ee903c

COLOR081: 色值 #f08500

COLOR082: 色值 #a9d92d

COLOR090: 色值 #dd6549

COLOR100: 色值 #cc463d

COLOR101: 色值 #cf3e36

COLOR102: 色值 #5E6671

image

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants