-
-
Notifications
You must be signed in to change notification settings - Fork 354
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
Reserving ticket with "Offline Payment" provider presents opaque error when on same day event occurs #1415
Comments
Hi @shanebrowncs , thanks for the detailed report. The "Bank Transfer" payment method was first implemented 10 years ago, when the quickest bank transfer (at least in Europe) could take 1+ working days to complete. I agree that we'd need something different to cover the new "instant bank transfer" offers. From your report, I see two different issues 1 - (bug) Payment page should be consistent with BankTransferManager. In your example, users should not see "Bank Transfer" as a available payment option 2 - (feature) support instant offline payment options We are observing a growing interest from organizers about different "instant payment" options available in different countries (see for example #1411 and #1384). WDYT? Would that be something you'd be interested in contributing? Thanks in advance |
Hi @cbellone, I agree with your assessment, allowing organizers to define custom offline payment methods would be a good solution. Yes, I would be interested in working on this. I would like to know more about what you have in mind regarding the customization features and general scope. Thanks, Shane |
That's great! 👍 Thank you! |
Describe the bug
Choosing the "Bank Transfer" payment option when reserving a ticket on the same day as the event occurs, the error
Payment processing error: An unexpected error has occurred. Please contact the event's organizers in order to get assistance.
appears.To Reproduce
Expected behavior
This issue appears to occur because the 'isActive' method on the 'BankTransferManager' payment provider returns false when the current day is the same day as the event [1]. When the endpoint at /reservation/{reservationId} attempts start processing the payment it can't find an active provider to fulfill the request and ends up displaying the opaque error above.
I have further thoughts on this behavior below but at bare minimum I don't think an attendee should be able to choose a non-active or invalid payment method. Otherwise at least presenting them with an error that explains what went wrong would be better.
From warnings posted in BankTransferManager this seems to be in a way by design to not allow same day payments:
accepting offline payments the same day is a very bad practice and should be avoided. Please set cash payment as payment method next time
. I have been using this feature to accept Interac E-Transfers which happen almost instantaneously. Using the cash payment option will not work for me as my events are online only. I can understand the perspective of the warning when managing a large event using bank-transfers with multi-day waiting periods. However, for my use-case of small online only events with near instantaneous transfers, accepting payments same day is not a big issue.Would you be open to one of the following changes?
I would be interested in working on a PR to address this issue. I just want to see first if you agree there is an issue here and if so what the solution should be.
Thank you!
Screenshots
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: