active_merchant icon indicating copy to clipboard operation
active_merchant copied to clipboard

Beanstream Gateway Improvements

Open lumiit-adam opened this issue 7 years ago • 8 comments

Integrated the IPP gateway into the Beanstream gatewayand use BeanstreamGateway as the unified entrance;

remove IPP gateway files;

Because the two gateways belong to the same company just different branch. So the developers from the company hope to make them an unified gateway to Shopify.

(Beansteam website: http://www.beanstream.com/)

lumiit-adam avatar Apr 23 '18 01:04 lumiit-adam

looks good to me, @WizardCloud please squash commit

elfassy avatar May 22 '18 16:05 elfassy

@elfassy squashed now

lumiit-adam avatar May 23 '18 05:05 lumiit-adam

@lumiit-adam can you post a link to the Beanstream docs in the description, thx

elfassy avatar May 29 '18 13:05 elfassy

Hi, @elfassy I have split the committed code, please have a look.

lumiit-adam avatar Jun 04 '18 02:06 lumiit-adam

I have changed code, please continue to review.

lumiit-adam avatar Jun 20 '18 00:06 lumiit-adam

@elfassy is there anything outstanding before this can be merged into Master?

chriswongipp avatar Jul 09 '18 02:07 chriswongipp

Finished squashing commit and waiting for final reviewing

lumiit-adam avatar Jul 19 '18 04:07 lumiit-adam

Any reason we must use the region option? Seems to me like it would be best to let the end user decide between BeanstreamDefautGateway and BeanstreamIppGateway . Each already provides the information as to where it can be used (https://github.com/activemerchant/active_merchant/pull/2813/files#diff-1db73f09eabc8b8a2be54b728f148ee9R7)

elfassy avatar Aug 21 '18 15:08 elfassy