Skip to content

Conversation

@8ma10s
Copy link
Contributor

@8ma10s 8ma10s commented Dec 26, 2022

PublicKeyCredential.isConditionalMediationAvailable() method is already available, mediation field's value "conditional" should also be available now.

isConditionalMediationAvailable(): Promise<boolean>;

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.

`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
@github-actions
Copy link
Contributor

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.

@8ma10s
Copy link
Contributor Author

8ma10s commented Dec 26, 2022

will update baseline

@saschanaz
Copy link
Contributor

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

A feature needs to be supported by two or more major browser engines to be included here, to make sure there is a good consensus among vendors: Gecko (Firefox), Blink (Chrome/Edge), and WebKit (Safari).

@saschanaz
Copy link
Contributor

Credential.isConditionalMediationAvailable is WebKit-only (which somehow returns true instead of false) but PublicKeyCredential.isConditionalMediationAvailable is also in Blink.

LGTM thus.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2023

There was an issue merging, maybe try again saschanaz. Details

@saschanaz
Copy link
Contributor

Oh actually, CLA?

@8ma10s
Copy link
Contributor Author

8ma10s commented Feb 3, 2023

@microsoft-github-policy-service agree

@8ma10s
Copy link
Contributor Author

8ma10s commented Feb 3, 2023

@saschanaz sorry I haven't checked back on this issue for a while. Just agreed. Please have a look at it again 🙏

@saschanaz
Copy link
Contributor

LGTM 👍

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

There was an issue merging, maybe try again saschanaz. Details

@saschanaz
Copy link
Contributor

Has some breaking change happened in GitHub side?

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

There was an issue merging, maybe try again saschanaz. Details

@saschanaz
Copy link
Contributor

Creating comments and merging
Merging (or commenting) failed:
Error: HttpError: Resource not accessible by integration
Error: Failed to merge

Huh?

@8ma10s
Copy link
Contributor Author

8ma10s commented Feb 6, 2023

weird. Maybe GitHub token has insufficient permissions...?

@saschanaz
Copy link
Contributor

LGTM

It has been working, not sure what's changed.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2023

There was an issue merging, maybe try again saschanaz. Details

@saschanaz
Copy link
Contributor

#1496 (comment) worked, so maybe it's this PR that is somehow broken? Can you try closing and reopening it?

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.

2 participants