-
Notifications
You must be signed in to change notification settings - Fork 6.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
添加 userId 到 PayOrderDO #555
base: master-jdk17
Are you sure you want to change the base?
Changes from all commits
466b6e9
0ebd8b5
643a50f
5786ff4
054323e
093fbc4
37b707e
c77ec84
4eb436c
d894a47
8235191
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,19 +32,19 @@ public class PayOrderDO extends BaseDO { | |
private Long id; | ||
/** | ||
* 应用编号 | ||
* | ||
* <p> | ||
* 关联 {@link PayAppDO#getId()} | ||
*/ | ||
private Long appId; | ||
/** | ||
* 渠道编号 | ||
* | ||
* <p> | ||
* 关联 {@link PayChannelDO#getId()} | ||
*/ | ||
private Long channelId; | ||
/** | ||
* 渠道编码 | ||
* | ||
* <p> | ||
* 枚举 {@link PayChannelEnum} | ||
*/ | ||
private String channelCode; | ||
|
@@ -53,7 +53,7 @@ public class PayOrderDO extends BaseDO { | |
|
||
/** | ||
* 商户订单编号 | ||
* | ||
* <p> | ||
* 例如说,内部系统 A 的订单号,需要保证每个 PayAppDO 唯一 | ||
*/ | ||
private String merchantOrderId; | ||
|
@@ -78,7 +78,7 @@ public class PayOrderDO extends BaseDO { | |
private Integer price; | ||
/** | ||
* 渠道手续费,单位:百分比 | ||
* | ||
* <p> | ||
* 冗余 {@link PayChannelDO#getFeeRate()} | ||
*/ | ||
private Double channelFeeRate; | ||
|
@@ -88,14 +88,18 @@ public class PayOrderDO extends BaseDO { | |
private Integer channelFeePrice; | ||
/** | ||
* 支付状态 | ||
* | ||
* <p> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 这种换行,不加哈。 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🤔 这个是 IDEA 格式化代码的时候自动加上去的... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 去掉哈。 ps:代码不要批量格式化。。。最好是自己写出来,就是标准的哈。 |
||
* 枚举 {@link PayOrderStatusEnum} | ||
*/ | ||
private Integer status; | ||
/** | ||
* 用户 IP | ||
*/ | ||
private String userIp; | ||
/** | ||
* 用户编号 | ||
*/ | ||
private Long userId; | ||
/** | ||
* 订单失效时间 | ||
*/ | ||
|
@@ -106,13 +110,13 @@ public class PayOrderDO extends BaseDO { | |
private LocalDateTime successTime; | ||
/** | ||
* 支付成功的订单拓展单编号 | ||
* | ||
* <p> | ||
* 关联 {@link PayOrderExtensionDO#getId()} | ||
*/ | ||
private Long extensionId; | ||
/** | ||
* 支付成功的外部订单号 | ||
* | ||
* <p> | ||
* 关联 {@link PayOrderExtensionDO#getNo()} | ||
*/ | ||
private String no; | ||
|
@@ -126,7 +130,7 @@ public class PayOrderDO extends BaseDO { | |
// ========== 渠道相关字段 ========== | ||
/** | ||
* 渠道用户编号 | ||
* | ||
* <p> | ||
* 例如说,微信 openid、支付宝账号 | ||
*/ | ||
private String channelUserId; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1、可以搞成非必传
2、建议 userType 也存储
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
userType
有必要记录么? 🤔除了管理员使用管理端测试支付的时候会产生零星的几个交易订单以为, 其他的应该都是会员产生的了吧?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
现有业务如此,但是有可能 admin 卖租户套餐,类似这种业务哈。