Skip to content
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

Authorize.Net Payment Failure Notices #141

Closed
jaswrks opened this issue May 12, 2014 · 10 comments
Closed

Authorize.Net Payment Failure Notices #141

jaswrks opened this issue May 12, 2014 · 10 comments

Comments

@jaswrks
Copy link
Contributor

jaswrks commented May 12, 2014

Why does s2Member not respond to individual payment failures? Well, because a single payment failure is not something that all site owners would want to force an EOT on, as the next payment might come through successfully. However, you can choose to receive emails for these and then suspend the recurring profile yourself, if you deem necessary. s2Member will respond to this.

That said, I agree there is some room for improvement here. In the case of PayPal Pro there is a setting that allows the site owner to control MAX_FAILED_PAYMENTS, and that seems like a good idea for Authorize.Net also. It's not something that Authorize.Net supports natively, but perhaps we can work to add this into a future release of s2Member Pro.

@jaswrks jaswrks added this to the Future Release milestone May 17, 2014
@jaswrks
Copy link
Contributor Author

jaswrks commented May 17, 2014

Bumping this to the "Future Release" Milestone.

@jaswrks jaswrks self-assigned this May 17, 2014
@clavaque
Copy link
Contributor

Maybe we should add a flag to users to know if the access is enabled or not, but without it actually being an EOT? So the subscription is still the same, but if the payment fails s2 can flag the account and suspend access until the payment goes through, and nothing changed in the subscription and no EOT happened, but he also didn't have free access during that time. And it could also include an email to the user.

I'm not sure if the notifications we get from the gateways would allow to do something like that. And it may not be an ideal solution either, but it could improve what we have right now.

@jaswrks
Copy link
Contributor Author

jaswrks commented May 29, 2014

add a flag to users to know if the access is enabled or not,

Are you saying that a failed payment should flag the account as having been disabled?; i.e. to deny them access starting on the first failed payment by flagging the account?

@clavaque
Copy link
Contributor

flag the account as having been disabled?

Not the account, but the access.

deny them access starting on the first failed payment

Yeah, something like that, until the payment happens so the access is restored, or the subscription is ended and EOT happens.

It's just an idea, not sure what you think about that. I'm basically trying to solve the problem of users with access although payment didn't happen.

@jaswrks
Copy link
Contributor Author

jaswrks commented May 29, 2014

Got it, thank you. OK, I'm going to think about this some more.

@jaswrks
Copy link
Contributor Author

jaswrks commented Jun 2, 2014

Moving this to the "Distant Release" Milestone.

@jaswrks jaswrks modified the milestones: Distant Release, Next Release Jun 2, 2014
@jaswrks
Copy link
Contributor Author

jaswrks commented Jun 25, 2014

@jaswrks
Copy link
Contributor Author

jaswrks commented Jun 25, 2014

The file we need to update is in s2Member Pro; where we listen to the Authorize.Net Silent Post URL connection. Authorize.Net sends s2Member Pro a notification when there is a failed payment. Referencing: s2member-pro/includes/classes/gateways/authnet/authnet-notify-in.inc.php

@jaswrks
Copy link
Contributor Author

jaswrks commented Jul 1, 2014

Moving this to the "Future Release" milestone.

@jaswrks jaswrks modified the milestone: Future Release Jul 1, 2014
@jaswrks jaswrks removed their assignment Jul 1, 2014
@jaswrks jaswrks self-assigned this Aug 6, 2014
@jaswrks
Copy link
Contributor Author

jaswrks commented Aug 6, 2014

Assigning this to myself.

jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 6, 2014
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants