Skip to content

[Editorial] Rename 'payment handler' to 'web-based payment handler' in the spec text#427

Open
stephenmcgruer wants to merge 1 commit intogh-pagesfrom
smcgruer-update-wording
Open

[Editorial] Rename 'payment handler' to 'web-based payment handler' in the spec text#427
stephenmcgruer wants to merge 1 commit intogh-pagesfrom
smcgruer-update-wording

Conversation

@stephenmcgruer
Copy link
Collaborator

closes #???

The following tasks have been completed:

  • web platform tests (link)
  • MDN Docs added (link)

Implementation commitment:

  • Safari (link to issue)
  • Chrome (link to issue)
  • Firefox (link to issue)
  • Edge (public signal)

</script>
</head>
<body data-cite="service-workers payment-method-id">
<body data-cite="service-workers payment-method-id payment-request">
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if this is the right way to do it, but this stopped respec from complaining that the [=payment handler=] reference didn't exist.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks correct to me. In the long term, migrating to xref would be preferable (per respec guidance), but that's a larger change unrelated to this patch. I think this is the right approach for now.

@stephenmcgruer
Copy link
Collaborator Author

I renamed almost all of the payment handler mentions to web-based payment handler, whether or not they were a link or just text. It is possible that some of them either (1) should be link-ified and/or (2) should be named back to just 'payment handler' (e.g., if they're talking about the generic concept). Feel free to suggest places that should be changed :)

@ianbjacobs
Copy link
Contributor

ianbjacobs commented Feb 18, 2026

(I deleted my comment; I left questions on a different pull request.)

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

Successfully merging this pull request may close these issues.

3 participants