This system is based on Paybox, but some settings change.

CommentFileSizeAuthor
#13 2652852-etransactions-13.patch1.04 KBjcisio
#10 2652852-etransactions-10.patch4.52 KBjcisio
#9 2652852-etransactions.patch3.68 KBjcisio
#3 e_transactions_mobile_support-2652852-3.patch1.49 KBArnaudDabouis
#2 e_transactions_support-2652852-1.patch761 bytesArnaudDabouis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arnaudlumini created an issue. See original summary.

ArnaudDabouis’s picture

ArnaudDabouis CreditAttribution: ArnaudDabouis at Lumini commented
FileSize
e_transactions_support-2652852-1.patch761 bytes
ArnaudDabouis’s picture

ArnaudDabouis CreditAttribution: ArnaudDabouis commented
FileSize
e_transactions_mobile_support-2652852-3.patch1.49 KB

Added mobile support.

jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
Status: Active » Needs work

Can you explain this patch please? What is e-Transactions and how this module is supposed to work with both?

Also, I've just added responsive support in #2857241: Paybox offsite - responsive mode (RWD). I saw the mobile support but didn't add it because I don't think of any case where you need "mobile only" payment but RWD does not work.

ArnaudDabouis’s picture

ArnaudDabouis CreditAttribution: ArnaudDabouis commented

E-transactions is a service from french bank Crédit Agricole that uses Paybox as a white label.

Maybe we could offer an option in the backend to switch from one service to another, or create another module that would only overwrite these constants, but it seems a little overkill.

So this patch could be meant to remain a patch for users using this particular service.

jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented

Thanks for the explanation. So I'll leave this issue. I think the easiest way is an option to replace Paybox domain with E-transaction domain for all URLs. Do you know any other service that uses white labelled Paybox?

ArnaudDabouis’s picture

ArnaudDabouis CreditAttribution: ArnaudDabouis commented

No, I don't know any other service using a white labelled Paybox.

jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
Assigned: Unassigned » jcisio
jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
Status: Needs work » Needs review
FileSize
2652852-etransactions.patch3.68 KB

Here is a proper patch to add E-Transactions support.

jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
FileSize
2652852-etransactions-10.patch4.52 KB

Add tests. I'd prefer integration test, but for now it's ok.

  • jcisio committed af6e96b on 7.x-1.x 
    Issue #2652852 by ArnaudDabouis, jcisio: Support for Crédit Agricole E-...
jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
Status: Needs review » Fixed
jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
Status: Fixed » Needs review
FileSize
2652852-etransactions-13.patch1.04 KB

I should have ran tests locally first.

jcisio’s picture

jcisio CreditAttribution: jcisio at Axess Open Web Services commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.