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(credentials): delete associated didCommMessages #870

Conversation

morrieinmaas
Copy link
Contributor

Signed-off-by: Moriarty [email protected]

closes #728

@morrieinmaas morrieinmaas requested a review from a team as a code owner June 16, 2022 10:15
@TimoGlastra TimoGlastra changed the title feat: delete associated didCommMessages when deleting credential records feat(credentials): delete associated didCommMessages Jun 16, 2022
Moriarty added 2 commits June 16, 2022 17:08
code review feedback

Signed-off-by: Moriarty <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jun 16, 2022

Codecov Report

Merging #870 (c5a602a) into main (398c958) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #870   +/-   ##
=======================================
  Coverage   87.56%   87.57%           
=======================================
  Files         463      463           
  Lines       11077    11083    +6     
  Branches     1743     1745    +2     
=======================================
+ Hits         9700     9706    +6     
  Misses       1377     1377           
Impacted Files Coverage Δ
...les/credentials/protocol/v1/V1CredentialService.ts 95.38% <100.00%> (ø)
...les/credentials/protocol/v2/V2CredentialService.ts 89.25% <100.00%> (ø)
.../modules/credentials/services/CredentialService.ts 100.00% <100.00%> (ø)

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 398c958...c5a602a. Read the comment docs.

@TimoGlastra TimoGlastra merged commit 1f8b6ab into openwallet-foundation:main Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deleting a credential record should delete the associated didcomm messages
3 participants