-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Digital Credentials: implement API using the Identity Document Services framework #46626
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
EWS run on previous version of this PR (hash 226140d) |
226140d
to
d79d437
Compare
EWS run on previous version of this PR (hash d79d437) |
d79d437
to
825e56e
Compare
EWS run on previous version of this PR (hash 825e56e) |
825e56e
to
21ab7ab
Compare
EWS run on previous version of this PR (hash 21ab7ab) |
21ab7ab
to
e14311f
Compare
EWS run on previous version of this PR (hash e14311f) |
e14311f
to
2e66740
Compare
EWS run on previous version of this PR (hash 2e66740) |
2e66740
to
8c4d2aa
Compare
EWS run on previous version of this PR (hash 8c4d2aa) |
8c4d2aa
to
4ee79ac
Compare
EWS run on previous version of this PR (hash 4ee79ac) |
4ee79ac
to
bcbc611
Compare
EWS run on previous version of this PR (hash bcbc611) |
bcbc611
to
cf0a49c
Compare
EWS run on previous version of this PR (hash cf0a49c) |
cf0a49c
to
153d380
Compare
EWS run on previous version of this PR (hash 153d380) |
153d380
to
6bdfefb
Compare
EWS run on previous version of this PR (hash 6bdfefb) |
6bdfefb
to
095d714
Compare
EWS run on previous version of this PR (hash 095d714) |
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.
LGTM with a couple of small comments, hopefully we can get @rr-codes's blessings with the WebKitSwift changes!
Source/WebKit/UIProcess/DigitalCredentials/WKDigitalCredentialsPicker.h
Outdated
Show resolved
Hide resolved
Source/WebKit/UIProcess/DigitalCredentials/WKDigitalCredentialsPicker.h
Outdated
Show resolved
Hide resolved
Source/WebKit/UIProcess/DigitalCredentials/WKDigitalCredentialsPicker.mm
Outdated
Show resolved
Hide resolved
095d714
to
180623a
Compare
EWS run on previous version of this PR (hash 180623a) |
@marcoscaceres minor typo in the commit title! |
180623a
to
8f67948
Compare
EWS run on previous version of this PR (hash 8f67948) |
8f67948
to
a96a7d6
Compare
EWS run on current version of this PR (hash a96a7d6) |
Going to safe merge queue this for now as it's been sitting for a while, but happy to make any additional changes as a followup. |
…es framework https://bugs.webkit.org/show_bug.cgi?id=294237 rdar://151879131 Reviewed by Abrar Rahman Protyasha. Implements the Digital Credentials API using the new Identity Document Services framework. On iOS, this allows the user to select digital credentials from application that serve as document providers. On macOS, credentiala selection cban be performed over CTAP with a compatible device by scanning a QR code. Canonical link: https://commits.webkit.org/296741@main
a96a7d6
to
c4e6770
Compare
Committed 296741@main (c4e6770): https://commits.webkit.org/296741@main Reviewed commits have been landed. Closing PR #46626 and removing active labels. |
c4e6770
a96a7d6