-
Notifications
You must be signed in to change notification settings - Fork 166
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
changing name of method to isUserVerifyingPlatformAuthenticatorAvailable
- Loading branch information
Showing
1 changed file
with
5 additions
and
5 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
25a35a9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is quite some name. Longest method on the web platform perhaps?
Wouldn't something like hasUserVerifyingAuthenticator() have worked too?
25a35a9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not wanting to judge here, because I didn't push for this, but the argument was that the two criteria for this method to work w/o being privacy-compromising was:
This is so that sites can use this method to decide if it's super-easy to help users enroll, or if they might need a more complex and guiding UX.
(@balfanz, @equalsJeffH, please fill in where I got this wrong. I don't see this discussion in the IRC logs, so I'm working from my notes.)