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

What can I do with a URL-based payment method identifier? #52

Closed
rsolomakhin opened this issue Jul 30, 2018 · 7 comments
Closed

What can I do with a URL-based payment method identifier? #52

rsolomakhin opened this issue Jul 30, 2018 · 7 comments

Comments

@rsolomakhin
Copy link

During a TAG review of Payment Handler, a reviewer raised concern that it was not clear what a user agent could do with a URL-based payment method identifier. For example, does a manifest live there?

Payment method manifest ingestion is something that we have defined in the payment method manifest spec, but we should also mention it in this spec.

cc @dbaron

@ianbjacobs
Copy link
Contributor

ianbjacobs commented Jul 30, 2018

Hi @rsolomakhin,

There used to be some text in the spec, but it was removed. However, I think at the time we did not have a Manifest spec, so maybe we could add mention to the spec, e.g.:

CURRENT:
Developers wanting to use a URL-based payment method identifier for a third party payment handler are encouraged to read the Payment Method Best Practice document.

NEW:
Developers wanting to use a URL-based payment method identifier for a third party payment handler are encouraged to read the Payment Method Manifest specification and the Payment Method Best Practice document.

Thoughts?

Ian

@rsolomakhin
Copy link
Author

That change sounds good. Perhaps we can be even more clear by adding this sentence at the end of the paragraph:

"These documents describe how to find the payment method manifest from a payment method identifier, allowing the user agent to discover installable payment apps, for example."

That wouldn't be too much, would it?

@stpeter
Copy link

stpeter commented Jul 30, 2018

Maybe s/installable/available/ (it's not clear to me what we mean by "installable" - does it imply that an app can be installed "in real time / behind the scenes" or that payment apps require an explicit installation step?)

@rsolomakhin
Copy link
Author

"available" sounds good.

@ianbjacobs
Copy link
Contributor

Updated proposal (deleting both installable and available):

Developers wanting to use a URL-based payment method identifier for a third party payment handler are encouraged to read the Payment Method Manifest specification and the Payment Method Best Practice document. Together, these documents describe how the owner of a payment method can manage the ecosystem of authorized payment handlers and how the browser can discover them (e.g., for just-in-time payment handler installation).

Ian

@stpeter
Copy link

stpeter commented Jul 30, 2018

@ianbjacobs I think we've gone astray with the last sentence. What about payment methods that don't have "owners" (say, Bitcoin) or that don't require authorization with a controlling entity (say, Interledger)? I liked what @rsolomakhin proposed, modulo the one word: "These documents describe how to find the payment method manifest from a payment method identifier, allowing the user agent to discover available payment apps, for example."

@ianbjacobs
Copy link
Contributor

I am not uncomfortable saying "owner of a payment method" in the sense that some entity controls the origin. I also like adding the other bit about manifests - that the entity controls the ecosystem of payment handlers.

Here's a simpler version that does not say owner:

Developers wanting to use a URL-based payment method identifier for a third party payment handler are encouraged to read the Payment Method Manifest specification and the Payment Method Best Practice document. Together, these documents describe how to manage the ecosystem of authorized payment handlers for a payment method, including just-in-time payment handler installation by the browser.

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

3 participants