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.

Sign up for GitHub

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

Jump to bottom

complete does not talk to the Payment App #129

Closed
AxelNennker opened this issue Apr 8, 2016 · 1 comment
Closed

complete does not talk to the Payment App #129

AxelNennker opened this issue Apr 8, 2016 · 1 comment
Assignees
@zkoch
Labels
Editorial Priority: High
Milestone
28 April

Comments

@AxelNennker
Copy link
Contributor

As discusses in issue #17 complete() does tell the UA that the payment is complete. It does not talk to the Payment App.
So step 5 should be removed.

Pass the value of <code>success</code> to the <a>Payment App</a> that accepted the payment request.
@adrianhopebailie adrianhopebailie added this to the Priority: High milestone Apr 10, 2016
@adrianhopebailie adrianhopebailie modified the milestone: Priority: High Apr 22, 2016
adrianhopebailie added a commit to adrianhopebailie/browser-payment-api that referenced this issue Apr 22, 2016
@adrianhopebailie
Removed parameter for complete()
79375d8 
Addresses w3c#129 by dropping the algorithm step that passes value to the
payment app. As the boolean passed to complete is not used for anything
else it has been dropped too.
@adrianhopebailie adrianhopebailie mentioned this issue Apr 22, 2016
Closed
@adrianhopebailie adrianhopebailie modified the milestone: Discuss on Call - 28 April Apr 28, 2016
@adrianhopebailie
Copy link
Collaborator

Addressed by #161

@adrianba adrianba mentioned this issue Sep 23, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zkoch zkoch

Labels
Editorial Priority: High
Projects
None yet
Milestone
28 April
Development

No branches or pull requests
3 participants
@adrianhopebailie @zkoch @AxelNennker