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

Signaling payment handler details errors #727

Closed
marcoscaceres opened this issue Jun 14, 2018 · 9 comments · Fixed by #752
Closed

Signaling payment handler details errors #727

marcoscaceres opened this issue Jun 14, 2018 · 9 comments · Fixed by #752

Comments

@marcoscaceres
Copy link
Member

For example, the card might have expired. There might be insufficient digits, etc...

@ianbjacobs
Copy link
Collaborator

To say a bit more: suppose the merchant gets back some payment credentials and those credentials don't work for some reason. The merchant might want to prompt the user again, but "minus the unacceptable credentials". Such information would need to be passed to the payment handler as well.

@marcoscaceres
Copy link
Member Author

The model in the spec is “the following things are erroneous: x, y, z.” But not “minus unacceptable”, which seems different to me.

@ianbjacobs
Copy link
Collaborator

I think there is a difference between:

  • Please correct this information, and
  • You can't use this, so the browser or payment handler should no longer present it as an option

Ian

@marcoscaceres
Copy link
Member Author

You can't use this, so the browser or payment handler should no longer present it as an option

I'm not in favor is this, tbh. Right now, everything maps to input fields.

@ianbjacobs
Copy link
Collaborator

We don't have this feature today and that has not been a show-stopper.

@marcoscaceres
Copy link
Member Author

Paying for TPAC today...

screenshot 2018-06-26 14 14 33

We can't do this today :)

@marcoscaceres
Copy link
Member Author

This bug should probably move to the basic card spec.

@marcoscaceres
Copy link
Member Author

Moved w3c/payment-method-basic-card#57

@marcoscaceres
Copy link
Member Author

Reopening, because this still needs infrastructure in this spec.

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

Successfully merging a pull request may close this issue.

2 participants