search

yansongda / pay

可能是我用过的最优雅的 Alipay/WeChat/Unipay/江苏银行 的支付 SDK 扩展包了
http://pay.yansongda.cn
MIT License
4.98k stars 1.02k forks source link

feat: 新增江苏银行e融支付 #1002

Closed carlin-rj closed 3 weeks ago

carlin-rj commented 4 weeks ago

新增江苏银行e融支付

codecov[bot] commented 4 weeks ago

Codecov Report

Attention: Patch coverage is 99.58848% with 1 line in your changes missing coverage. Please review.

Project coverage is 97.49%. Comparing base (df334dd) to head (052092b).

Files Patch % Lines
src/Plugin/Jsb/Pay/Scan/PayPlugin.php 92.30% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1002 +/- ## ============================================ + Coverage 97.40% 97.49% +0.08% - Complexity 909 952 +43 ============================================ Files 347 361 +14 Lines 6213 6456 +243 ============================================ + Hits 6052 6294 +242 - Misses 161 162 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

yansongda commented 4 weeks ago

麻烦帮忙看看刚刚另外两个点，其他看着应该没问题了，改完之后这周我 PR 进来。

PS： 如果有时间的话，麻烦帮忙把文档也补充下

carlin-rj commented 4 weeks ago

麻烦帮忙看看刚刚另外两个点，其他看着应该没问题了，改完之后这周我 PR 进来。

PS： 如果有时间的话，麻烦帮忙把文档也补充下

你说的问题都已解决

yansongda commented 3 weeks ago

@carlin-rj 抱歉打扰，请问方便给下江苏银行的文档么？我这边正在准备合代码，现在正拉下来仔细系统的审核一下，发现 scan 目录下的各个文件都没有相应的文档注释

yansongda commented 3 weeks ago

我优化了一下代码，麻烦看看为啥那个单测没有过，我感觉你以前写的那个单测有问题

carlin-rj commented 3 weeks ago

我优化了一下代码，麻烦看看为啥那个单测没有过，我感觉你以前写的那个单测有问题 不好意思，是我的疏忽。已经修改

carlin-rj commented 3 weeks ago

@carlin-rj 抱歉打扰，请问方便给下江苏银行的文档么？我这边正在准备合代码，现在正拉下来仔细系统的审核一下，发现 scan 目录下的各个文件都没有相应的文档注释

e融支付.zip