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

Add compat data for secure payment confirmation #21460

Merged
merged 4 commits into from
Dec 8, 2023
Merged

Conversation

wbamberg
Copy link
Contributor

@wbamberg wbamberg commented Dec 4, 2023

Adds compat data for secure payment confirmation.

Supersedes #20583, using the same data as in that PR, but as additions to https://developer.mozilla.org/en-US/docs/Web/API/CredentialsContainer/create (for the WebAuth extension) and https://developer.mozilla.org/en-US/docs/Web/API/PaymentRequest/PaymentRequest (for the new payment method.

I've also added links to docs for all the other WebAuth extensions. The data for minPinLength was doing that using mdn_url but that didn't work because fragments seem to get ignored there (?).

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Dec 4, 2023
@ianbjacobs
Copy link

cc @stephenmcgruer

Thank you so much, @wbamberg! I will have a look.

@stephenmcgruer
Copy link

LGTM, thanks!

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

The data for minPinLength was doing that using mdn_url but that didn't work because fragments seem to get ignored there (?).

hm, the way minPinLength was doing that is the more consistent way in BCD. I would suggest you have mdn_url and description for all entries. That way caniuse will render:
"See full reference on MDN Web Docs."

If yari doesn't render the full url, then that's a bug in yari.

@wbamberg
Copy link
Contributor Author

wbamberg commented Dec 8, 2023

Ah, thanks for letting me know! I updated this PR and filed mdn/yari#10152.

api/PaymentRequest.json Outdated Show resolved Hide resolved
@wbamberg wbamberg requested a review from Elchi3 December 8, 2023 18:04
api/PaymentRequest.json Outdated Show resolved Hide resolved
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Thanks!

@Elchi3 Elchi3 merged commit b77fd7b into mdn:main Dec 8, 2023
6 checks passed
@ianbjacobs
Copy link

My sincere thanks to you all for helping get this done!

@wbamberg
Copy link
Contributor Author

wbamberg commented Dec 8, 2023

My sincere thanks to you all for helping get this done!

You're welcome, and thanks for your patience with our pedantry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants