Repository navigation
Conversation
|
@Bkizzy16 This PR now has merge conflicts with git fetch upstream # upstream = https://github.com/DocForum/docforum-escrow.git
git rebase upstream/main
# resolve the conflicts, re-run the tests, then:
git push --force-with-leaseThe usual culprits are shared docs like Automated message. It won't repeat unless the PR conflicts again after your next push. |
|
Hi @afeezorobsco-cyber — I updated PR #263 onto current main and resolved the roadmap conflict while preserving the authorization-tree test entry. GitHub reports the branch mergeable; the latest malware scan passes, and the previous contract/SDK test checks passed. Please rerun/approve full CI on the updated head. |
|
@Bkizzy16 After merging #209 (protocol fee) added two parameters to |
|
@Bkizzy16 This PR now has merge conflicts with git fetch upstream # upstream = https://github.com/DocForum/docforum-escrow.git
git rebase upstream/main
# resolve the conflicts, re-run the tests, then:
git push --force-with-leaseThe usual culprits are shared docs like Automated message. It won't repeat unless the PR conflicts again after your next push. |
Summary
create_escrowpayer authorization to its exact invocation tree, including the nested tokentransfer.releaseandrefundtests, including wrong-caller cases, and update Soroban auth snapshots.This is an incremental contribution to #105, not the full issue completion: other escrow test modules still contain
mock_all_auths()and need conversion before the issue can be closed.Validation
cargo test --manifest-path contracts/escrow/Cargo.toml --test create_escrow— passed (7 tests).cargo test --manifest-path contracts/escrow/Cargo.toml --test release_refund— passed (7 tests).git diff --check— passed.Part of #105.