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

BasicCardResponse: cardholderName should be optional. #134

Closed
Torlus opened this issue Apr 10, 2016 · 6 comments
Closed

BasicCardResponse: cardholderName should be optional. #134

Torlus opened this issue Apr 10, 2016 · 6 comments

Comments

@Torlus
Copy link

Torlus commented Apr 10, 2016

I see at least two cases where this field cannot be returned:

  • In France, it is forbidden to provide such information, due to the CNIL rules about personal data and rights.
  • Visa, MasterCard, and others provide Anonymous Prepaid Cards, making this information irrelevant.
@mattsaxon
Copy link
Contributor

In my PR #114, I have indeed showed this as optional.

@adrianhopebailie
Copy link
Collaborator

Addressed in PR #114

@mattsaxon
Copy link
Contributor

Adrian, that PR has not been accepted yet and I've in fact closed it awaiting resolution of PR 133 which changes the shape of how PR 114 will be done.

@mattsaxon mattsaxon reopened this Apr 26, 2016
@adrianhopebailie
Copy link
Collaborator

Sorry, my mistake. I saw that @adrianba had merged #131 which included the same change but now see he picked only the one specific commit.

@mattsaxon
Copy link
Contributor

PR #176 submitted to address

@adrianba
Copy link
Contributor

adrianba commented May 5, 2016

Fixed in 9c32ebc.

@adrianba adrianba closed this as completed May 5, 2016
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

No branches or pull requests

5 participants