Skip to content

Move v1 response handling to V1Context#469

Merged
spacebear21 merged 1 commit intopayjoin:masterfrom
DanGould:split-v1-psbt-contexts
Jan 8, 2025
Merged

Move v1 response handling to V1Context#469
spacebear21 merged 1 commit intopayjoin:masterfrom
DanGould:split-v1-psbt-contexts

Conversation

@DanGould
Copy link
Contributor

@DanGould DanGould commented Jan 8, 2025

The V1-specific error handling was being done in PsbtContext even though the PsbtContext should not know about V1-related json errors.

The V1-specific error handling was being done in PsbtContext even though
the PsbtContext should not know about V1-related json errors.
@DanGould DanGould requested a review from spacebear21 January 8, 2025 03:28
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 12663535531

Details

  • 28 of 30 (93.33%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 61.178%

Changes Missing Coverage Covered Lines Changed/Added Lines %
payjoin/src/send/v1.rs 25 27 92.59%
Totals Coverage Status
Change from base Build 12653835492: 0.02%
Covered Lines: 2928
Relevant Lines: 4786

💛 - Coveralls

@spacebear21 spacebear21 merged commit fc7dc75 into payjoin:master Jan 8, 2025
6 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