This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

Correct payment_instrument_id and payment_type values in processor definitions. #67

Closed

artfulrobot opened this issue Jan 31, 2020 · 1 comment

Milestone

Owner

artfulrobot commented Jan 31, 2020

It seems that both the Payment Processor Type and the Payment Processor records were not set correctly historically:

payment_type is defined as 1 = credit card, 2 = direct debit. So we should be using 2.
payment_instrument_id seems to have defaulted to 1 (i.e. credit card unless you have a non-standard install), but should point to the instrument created for GoCardless.

I don't think this will have affected anything but it should be corrected.

The text was updated successfully, but these errors were encountered:

artfulrobot pushed a commit that referenced this issue


          WIP on fixing #67

589f92d

artfulrobot added this to the 1.10 milestone

artfulrobot mentioned this issue

Payment Method is null for recurring GoCardless payment #63

Closed

artfulrobot modified the milestones: 1.10, 1.9.2

Owner Author

artfulrobot commented Feb 6, 2020

Closing as fixed in 1.9.2beta2, will make 1.9.2

artfulrobot closed this as completed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.