-
Notifications
You must be signed in to change notification settings - Fork 204
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
fix(credentials): default for credentials in exchange record #816
fix(credentials): default for credentials in exchange record #816
Conversation
Signed-off-by: Mike Richardson <[email protected]>
Signed-off-by: Mike Richardson <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #816 +/- ##
==========================================
- Coverage 87.70% 87.70% -0.01%
==========================================
Files 438 438
Lines 10823 10822 -1
Branches 1892 1891 -1
==========================================
- Hits 9492 9491 -1
Misses 1269 1269
Partials 62 62
Continue to review full report at Codecov.
|
packages/core/src/modules/credentials/repository/CredentialExchangeRecord.ts
Outdated
Show resolved
Hide resolved
…hangeRecord.ts Signed-off-by: Timo Glastra <[email protected]>
@NB-MikeRichardson it looks like the tests are failing now but only because the |
Signed-off-by: Mike Richardson <[email protected]>
Head branch was pushed to by a user without write access
Signed-off-by: Mike Richardson <[email protected]>
The tests are still failing. No idea why. |
Signed-off-by: Timo Glastra <[email protected]>
@NB-MikeRichardson ran it locally and worked fine for me, there's probably something broken in your environment? I've pushed changes and it's ready to merge now! |
No description provided.