Opened 3 years ago
Last modified 22 months ago
#1078 assigned enhancement
Create a basic bank transfer payment module
Reported by: | pedersen | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | v2.1 |
Component: | EPayment | Version: | 0.98-dev |
Keywords: | Cc: |
Description
To allow bank transfer as a payment option, a basic bank transfer module has been created. It will generate a page and email with the necessary account information.
Concerning the attached patches:
- is the file htdocs/bankTransfer.py really necessary? I modeled it after the files for paypal and co.
- the second patch adds a hook to inject text from the plugin to paymentDetails.
Attachments (5)
Change History (17)
Changed 3 years ago by pedersen
comment:1 Changed 3 years ago by pferreir
Thanks a lot for the patch.
May I ask what is the purpose of the module? Is it only to show the account information?
How different is it from adding this information to the "Payment details" field in the E-payment module config?
comment:2 Changed 3 years ago by pferreir
- Status changed from new to infoneeded_new
comment:3 Changed 3 years ago by pedersen
The main difference is: This module has access to the full user information and can deleiver e.g.the full subject with a personalized ID for tracking the bank transfer. Now a user can select bank transfer explizitly on the payment page and will get a personalized display.
I plan to enhance the module to be able to also generate a pdf-invoice as attachment to the mail, so this will maybe pave a way for #751.
comment:4 Changed 3 years ago by jbenito
- Milestone set to v1.0
- Status changed from infoneeded_new to new
comment:5 Changed 3 years ago by jbenito
- Status changed from new to awaiting_merge
comment:6 Changed 3 years ago by jbenito
- Status changed from awaiting_merge to merging
comment:7 Changed 3 years ago by jbenito
- Milestone changed from v1.0 to v1.1
- Status changed from merging to assigned
Hi Pedersen,
Thank you very much for your plugin. I was having a look at your contribution and it is a very good idea.
On the other hand, we have the feeling that creating an epayment plug-in for that is a bit too big. We definitely see the need and we'd like to propose a different solution:
In the Epayment tab for a conference:
http://indico.cern.ch/ihelp/html/UserGuide/Conferences.html?highlight=epayment#e-payment
There is a field named "Detail of payment". Whatever somebody writes there, it shows up in the page where one can choose the e-payment system (last step in the registration process).
We could add there, the possibility to insert "tags" and one tag could be "UniqueID"; in the same way as it's done in other parts of Indico (e.g.: http://indico.cern.ch/ihelp/html/_images/confreg14.png). Then, an event organizer could write a text like:
You can pay using our e-payment system (see "Pay now" button below) or by bank transfer using the following details: Subject: {UniqueID} Bank account: XXXXX Bank: YYYY''
Indico would replace {UniqueID} by the real unique ID for each registrant.
Even more, we could rename "Detail of payment" to "Bank transfer details" and directly let Indico handle the different text choices and display of options. Indico would directly display either:
- You can pay by bank transfer or credit card, or
- You can only pay by bank transfer, here the details..., or
- You can only pay by credit card from the following button...
What do you think?
comment:8 Changed 3 years ago by pedersen
Yes, the possibility to add tags to the details screen is a good idea. Your suggestion sounds really promising to me.
With the plugin I really aimed for the possibility to attach a pdf invoice with all necessary details and generate an email with all details.
comment:9 Changed 3 years ago by jbenito
I see, as you have mentioned before, we would like to be able to generate bills (#751) directly from the list of registrants. Right now you can click and get a PDF with badges for instance. We could do the same for bills, we already have all the information (unique id, how much money they need to pay, etc) that we need.
If you agree on the "tags" idea and you would have the time, we would be more than happy to integrate it. Thanks again for your time!
comment:10 Changed 2 years ago by pedersen
The changes are now available via git from git://trac.frm2.tum.de/home/repos/git/frm2/webserver/indico.git as well in the brach master_email
comment:11 Changed 2 years ago by jbenito
- Milestone changed from v1.1 to v1.2
comment:12 Changed 22 months ago by jbenito
- Milestone changed from v1.2 to v1.3
Main patch