The issue remains unresolved until we update the version of graphql-php. I am uploading a patch here that can be applied to graphql-php (not this module). The linked issue at #3 contains the changes along with additional test coverage, but the patch no longer applies.
Some styling fixes
thenchev → created an issue.
thenchev → created an issue.
thenchev → created an issue.
thenchev → made their first commit to this issue’s fork.
Thanks, looks good to me.
Should be good to go.
thenchev → made their first commit to this issue’s fork.
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.
thenchev → made their first commit to this issue’s fork.
andrei.vesterli → credited thenchev → .
thenchev → created an issue.
Thanks Saf, I didn't do any testing in case we have any deprecated code. Let me know if there are any issues.
thenchev → created an issue.
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.
thenchev → made their first commit to this issue’s fork.
re-roll
Just a re-roll, the patch was not applying anymore for me. No changes except removed some unused use statements
just a reroll.
thenchev → created an issue.