LexikPayboxBundle icon indicating copy to clipboard operation
LexikPayboxBundle copied to clipboard

Published 20 hours ago • verified publisher icon lexik

Handle multiple paybox accounts

Open sarramegnag opened this issue 7 years ago • 3 comments

Greetings,

Huge PR related to https://github.com/lexik/LexikPayboxBundle/issues/89. Unfortunatly I think it would be too hard to handle backward compatibility.

Thanks for your review and feel free to ask if you have any questions.

Regards,

Guillaume

sarramegnag avatar Nov 20 '17 15:11 sarramegnag

@spike31 Is this PR worth a v3 tag ?

(i've changed jobs now and i won't be able to continue support on this plugin unfortunately but i think this PR is worth some investigation)

James

acidjames avatar Jan 21 '19 09:01 acidjames

Just for the information, we're using my fork in production since one year and a half. No problem until now with direct payments (we're not using recurring payments).

sarramegnag avatar Jan 21 '19 09:01 sarramegnag

Well... Handle multiple accounts is not common.

I'm not sure this is a feature we would like to see.

Or create an other bundle with a dps on that bundle, i don't know. Maybe we can mention your fork on the documentation, just in case

gilles-g avatar Jan 21 '19 12:01 gilles-g