Account created on 16 April 2011, about 13 years ago
#

Merge Requests

Recent comments

πŸ‡·πŸ‡ΈSerbia thenchev

thenchev β†’ made their first commit to this issue’s fork.

πŸ‡·πŸ‡ΈSerbia thenchev

Should be good to go.

πŸ‡·πŸ‡ΈSerbia thenchev

thenchev β†’ made their first commit to this issue’s fork.

πŸ‡·πŸ‡ΈSerbia thenchev

Looks good. We want to support for both versions of the Address module now but later I think we need a new major version of Address Decoupled that just supports 2.0. Over time 2.[something] will be too different.

πŸ‡·πŸ‡ΈSerbia thenchev

thenchev β†’ made their first commit to this issue’s fork.

πŸ‡·πŸ‡ΈSerbia thenchev

Thanks Saf, I didn't do any testing in case we have any deprecated code. Let me know if there are any issues.

πŸ‡·πŸ‡ΈSerbia thenchev

We just need to make sure the payment gateways we use in the future have it.

It looks like the bin number is just the last 6 (or 4) digits of the credit card. Based on Paytrace docs. We might need to introduce a fallback here for other payment gateways that don't have that included and just get the portion of the credit card. This can be done later.

πŸ‡·πŸ‡ΈSerbia thenchev

Just a re-roll, the patch was not applying anymore for me. No changes except removed some unused use statements

Production build 0.69.0 2024