-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat(app): Add support for custom scope in verfication flow #666
Conversation
9a98651
to
1d61612
Compare
demo/app/ios/Runner/OpenID4VP.swift
Outdated
try initiatedInteraction.presentCredential(selectedCredentials) | ||
|
||
if customScopes.count != 0 { | ||
let opts2 = Openid4vpNewPresentCredentialOpts()?.addScopeClaim(customScopes.keys.first, claimJSON: (customScopes.values.first as! String)) |
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.
You need to add all custom scopes from customScopes: Dictionary
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.
addressed
1d61612
to
10b1040
Compare
10b1040
to
c8a4e21
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #666 +/- ##
==========================================
+ Coverage 89.09% 89.10% +0.01%
==========================================
Files 103 103
Lines 4146 4150 +4
==========================================
+ Hits 3694 3698 +4
Misses 308 308
Partials 144 144 ☔ View full report in Codecov by Sentry. |
c8a4e21
to
5f1b8e7
Compare
Signed-off-by: Talwinder kaur <[email protected]>
5f1b8e7
to
608d777
Compare
Kudos, SonarCloud Quality Gate passed! |
Signed-off-by: Talwinder kaur [email protected]