-
Notifications
You must be signed in to change notification settings - Fork 456
mediation: "conditional" should be available now
#1454
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
Conversation
`PublicKeyCredential.isConditionalMediationAvailable()` method is already available, `mediation` field's value `"conditional"` should also be available now. https://github.com/microsoft/TypeScript-DOM-lib-generator/blob/0337998b669df62d10f3acb96cc2f7c2382bbfda/baselines/dom.generated.d.ts#L11245
|
Thanks for the PR! This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged. |
|
will update baseline |
|
Only Safari supports it, and that's not enough: https://github.com/microsoft/TypeScript-DOM-lib-generator#why-is-my-fancy-api-still-not-available-here
|
|
LGTM thus. |
|
There was an issue merging, maybe try again saschanaz. Details |
|
Oh actually, CLA? |
|
@microsoft-github-policy-service agree |
|
@saschanaz sorry I haven't checked back on this issue for a while. Just agreed. Please have a look at it again 🙏 |
|
LGTM 👍 |
|
There was an issue merging, maybe try again saschanaz. Details |
|
Has some breaking change happened in GitHub side? LGTM |
|
There was an issue merging, maybe try again saschanaz. Details |
Huh? |
|
weird. Maybe GitHub token has insufficient permissions...? |
|
LGTM It has been working, not sure what's changed. |
|
There was an issue merging, maybe try again saschanaz. Details |
|
#1496 (comment) worked, so maybe it's this PR that is somehow broken? Can you try closing and reopening it? |
PublicKeyCredential.isConditionalMediationAvailable()method is already available,mediationfield's value"conditional"should also be available now.TypeScript-DOM-lib-generator/baselines/dom.generated.d.ts
Line 11245 in 0337998
There was a previous issue that proposed the same thing, but it was closed because 'isConditionalMediationAvailable' was not available.
#1391
However, it's now available in both MDN document and this repo's
baselines/file.