-
Notifications
You must be signed in to change notification settings - Fork 183
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
Editorial: Call out payment handlers in abort() #679
Conversation
Per 29 Jan 2018 Payment Handler API editor discussion [1], acknowledge connection to payment handlers in abort() method. Relates to Issue #476. [1] https://www.w3.org/2018/01/29-apps-minutes#item02
Argh. Not sure why travis failing here. @deniak are you able to help? |
The travis config file was deleted as part of CR, because we are not auto-publishing. It’s safe to ignore the error. We should add a default config that just exits gracefully. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tiny change needed. Can you kindly also change the commit message to be prefixed with “editorial: “.
index.html
Outdated
<li>Try to abort the current user interaction and close down any | ||
remaining user interface. | ||
<li>Try to abort the current user interaction with the payment | ||
handler and close down any remaining user interface. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just need <a>
around payment handler.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Per 29 Jan 2018 Payment Handler API editor discussion [1], acknowledge
connection to payment handlers in abort() method.
Relates to Issue #476.
[1] https://www.w3.org/2018/01/29-apps-minutes#item02
Preview | Diff