Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added M_INFO field #6

Merged
merged 2 commits into from
Jul 16, 2024
Merged

Added M_INFO field #6

merged 2 commits into from
Jul 16, 2024

Conversation

rosen4o
Copy link
Contributor

@rosen4o rosen4o commented Jul 15, 2024

According to version 5.0 of the documentation from 12 Aug 2024 M_INFO field is required for transactions with VISA cards

@veneliniliev
Copy link
Owner

Thank you for the PR.

  • According to the documentation, the data should be in this format: {"email":"[email protected]","cardholderName":"CARDHOLDER NAME","mobilePhone":{"cc":"359","subscriber":"8939999888"}}. At least this is how the example is given. You've mentioned it incorrectly in README.md.
  • Will there be any issues if this field is not filled in with the current version?
  • The tests are failing and we will need to update them to accommodate this parameter.

We need to resolve these issues before we can merge the PR.

@rosen4o
Copy link
Contributor Author

rosen4o commented Jul 15, 2024

Have added backward compability and updated information in the README file

@veneliniliev veneliniliev merged commit deffffa into veneliniliev:master Jul 16, 2024
@veneliniliev
Copy link
Owner

@rosen4o Thank you for the PR. It has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants