Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

fix: holder cred-get-list returns only holder creds #120

Merged
merged 2 commits into from
Jan 17, 2022
Merged

fix: holder cred-get-list returns only holder creds #120

merged 2 commits into from
Jan 17, 2022

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Jan 17, 2022

Signed-off-by: Daniel Bluhm dbluhm@pm.me

@dbluhm
fix: holder cred-get-list returns only holder creds
0040154 
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
@dbluhm
Copy link
Contributor Author

dbluhm commented Jan 17, 2022

cc: @mepeltier
Turns out this was an issue in the plugin rather than in the toolbox

cjhowland
cjhowland previously approved these changes Jan 17, 2022
View reviewed changes
Copy link
Contributor

@cjhowland cjhowland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@dbluhm dbluhm enabled auto-merge January 17, 2022 22:12
@dbluhm
fix: post filter vs tag filter and int tests
b4ce3a2 
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
@dbluhm dbluhm merged commit 1501b1f into hyperledger-archives:main Jan 17, 2022
@dbluhm dbluhm deleted the fix/holder-get-cred-list branch January 18, 2022 02:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cjhowland cjhowland cjhowland approved these changes

@TheTechmage TheTechmage Awaiting requested review from TheTechmage

@burdettadam burdettadam Awaiting requested review from burdettadam

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dbluhm @cjhowland