-
Notifications
You must be signed in to change notification settings - Fork 8
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
bug/swtch-3035 Not valid VC are verified #761
Conversation
src/modules/claims/claims.service.ts
Outdated
} | ||
issuerVerified = issuerVerified && issuerVerificationResult.verified; |
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.
issuerVerified
is also used as result of status verification, so the final issuerVerified
is boolean AND
of both checks. Perhaps this checks should be separated
The CI is not passing, but it is something wrong with setting up ipfs as locally tests are passing if I run them separately. The exception is |
3f8fd29
to
7357300
Compare
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.
A small suggestion, but seems good to me
Co-authored-by: John Henderson <[email protected]>
🎉 This PR is included in version 8.1.1-alpha.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 8.1.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Checks issuer verification result
Contributor checklist