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

fix: their did doc not ours #436

Merged

Conversation

TimoGlastra
Copy link
Contributor

Was filtering based on our did doc, not theirs....

Signed-off-by: Timo Glastra <timo@animo.id>
@TimoGlastra TimoGlastra requested a review from a team as a code owner August 21, 2021 21:18
@codecov-commenter
Copy link

Codecov Report

Merging #436 (56c0bae) into main (163cda1) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #436      +/-   ##
==========================================
- Coverage   86.07%   86.06%   -0.02%     
==========================================
  Files         252      252              
  Lines        5394     5395       +1     
  Branches      797      797              
==========================================
  Hits         4643     4643              
- Misses        751      752       +1     
Impacted Files Coverage Δ
...ckages/core/src/modules/routing/RecipientModule.ts 63.39% <0.00%> (-0.42%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 163cda1...56c0bae. Read the comment docs.

@TimoGlastra TimoGlastra merged commit 0226609 into openwallet-foundation:main Aug 25, 2021
@TimoGlastra TimoGlastra deleted the fix/their-did-doc branch August 25, 2021 10:24
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.

3 participants