-
Notifications
You must be signed in to change notification settings - Fork 265
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
对 PR #29 的讨论 #30
Comments
@Varorbc 我写的那个 |
感觉还是用lazy吧,你觉得呢? |
为了支持 多租户 或者 多支付配置, 现在的想法是:
另外将 支付的组件抽象为: 这样的好处
丢失的功能
|
多租户功能支持了吗? |
@viewtance 一直是支持的 |
对于 PR #29 的讨论
讨论的方面:
方案1: 将 Target 挪到独立的命名空间下, 如
PaySharp.Target.*
(PaySharp.Target.AspNetCore)方案2: 将 支付机构的实现挪到独立的命名空间下, 如
PaySharp.Authorities.*
(PaySharp.Authorities.Alipay)The text was updated successfully, but these errors were encountered: