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

Clear both KEL and TEL escrows; list all events in escrow, or their counts #894

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

kentbull
Copy link
Contributor

This backports kli escrow clear and kli escrow list into the 1.1.x release branch.
I also added clearing for every escrow I could find as there seemed to be some missing from the 1.2.x branch. I upgraded the kli escrow list command to show a count of all existing escrows.

@kentbull kentbull force-pushed the escrow-clear-1-1x-releases branch from 4b87974 to eb185f0 Compare November 22, 2024 20:40
@kentbull kentbull requested a review from m00sey November 22, 2024 20:41
Copy link
Contributor

@daidoji daidoji left a comment

Choose a reason for hiding this comment

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

Mostly just reviewing for curiosity's sake but looks good to me. Maybe a few comments of suggestions but nothing crucial.

src/keri/app/cli/commands/escrow/clear.py Show resolved Hide resolved
src/keri/app/cli/commands/escrow/list.py Show resolved Hide resolved
src/keri/app/cli/commands/escrow/list.py Outdated Show resolved Hide resolved
src/keri/app/cli/commands/escrow/list.py Show resolved Hide resolved
src/keri/db/dbing.py Show resolved Hide resolved
@kentbull kentbull force-pushed the escrow-clear-1-1x-releases branch from eb185f0 to 92b2a6c Compare November 25, 2024 03:49
@m00sey m00sey merged commit 5cc5e2a into WebOfTrust:v1.1.22 Nov 25, 2024
6 checks passed
@kentbull kentbull deleted the escrow-clear-1-1x-releases branch November 25, 2024 12:52
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.

3 participants