Skip to content

Add --session-id to resume command to allow resuming single sessions - #1705

Merged
benalleng merged 1 commit into
payjoin:masterfrom
benalleng:resume-session-ids
Jul 2, 2026
Merged

benalleng merged 1 commit into
payjoin:masterfrom
benalleng:resume-session-ids

Conversation

@benalleng

Copy link
Copy Markdown
Collaborator

Adds the ability to resume single sessions instead of all sessions at once.

usage --session-id {session_id} to only resume that individual session. will fail if the session is already closed or not existent.

Coded with Deepseek v4

Pull Request Checklist

Please confirm the following before requesting review:

@benalleng
benalleng requested a review from spacebear21 July 2, 2026 19:48
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 28617298532

Coverage decreased (-0.02%) to 85.725%

Details

  • Coverage decreased (-0.02%) from the base build.
  • Patch coverage: 5 uncovered changes across 3 files (6 of 11 lines covered, 54.55%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
payjoin-cli/src/app/v2/mod.rs 3 1 33.33%
payjoin-cli/src/main.rs 6 4 66.67%
payjoin-cli/src/app/v1.rs 1 0 0.0%
Total (4 files) 11 6 54.55%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 15321
Covered Lines: 13134
Line Coverage: 85.73%
Coverage Strength: 355.68 hits per line

💛 - Coveralls

@spacebear21 spacebear21 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

concept ACK and utACK

@benalleng
benalleng merged commit f60825c into payjoin:master Jul 2, 2026
13 checks passed
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