Skip to content
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: add support for verifier logo uri #1378

Merged
merged 1 commit into from
Aug 4, 2023

Conversation

aholovko
Copy link
Contributor

@aholovko aholovko commented Aug 4, 2023

No description provided.

@cla-bot cla-bot bot added the cla-signed label Aug 4, 2023
@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (564094e) 87.44% compared to head (7c68c81) 87.45%.
Report is 1 commits behind head on main.

❗ Current head 7c68c81 differs from pull request most recent head 6c24e54. Consider uploading reports for the commit 6c24e54 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1378   +/-   ##
=======================================
  Coverage   87.44%   87.45%           
=======================================
  Files         154      154           
  Lines        9274     9275    +1     
=======================================
+ Hits         8110     8111    +1     
  Misses        768      768           
  Partials      396      396           
Files Changed Coverage Δ
pkg/service/oidc4vp/oidc4vp_service.go 91.23% <100.00%> (+0.02%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Andrii Holovko <andriy.holovko@gmail.com>
@aholovko aholovko force-pushed the verifier_logo_uri branch from 7c68c81 to 6c24e54 Compare August 4, 2023 08:22
@fqutishat fqutishat merged commit 34111d0 into trustbloc:main Aug 4, 2023
@aholovko aholovko deleted the verifier_logo_uri branch August 4, 2023 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants