Skip to content

fix: escrow sponsor/status index, dead stellar config removal, releasePartial/refund and SorobanClientService test coverage - #208

Merged
chonilius merged 1 commit into
MergeFi:mainfrom
hardcordev:fix/escrow-index-tests-config-94-95-96-97
Aug 26, 2026
Merged

fix: escrow sponsor/status index, dead stellar config removal, releasePartial/refund and SorobanClientService test coverage#208
chonilius merged 1 commit into
MergeFi:mainfrom
hardcordev:fix/escrow-index-tests-config-94-95-96-97

Conversation

@hardcordev

Copy link
Copy Markdown
Contributor

Summary

Added the missing composite index on escrows(sponsorId, status), removed five dead Stellar config fields that were documented but never read, and filled two unit-test gaps: direct coverage for EscrowService.releasePartial()/refund() and a dedicated spec for SorobanClientService.

Changes

Issues

Resolves #97
Resolves #96
Resolves #95
Resolves #94

Verification

Verified by manual code review of the full diff against the service implementations, entity/migration conventions, jest config, and the existing stellar-sdk mock. cargo build and cargo test were not run.

…er releasePartial/refund and SorobanClientService
@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@hardcordev Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

@hardcordev is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

@chonilius
chonilius merged commit 7306f01 into MergeFi:main Aug 26, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment