Skip to content

done#409

Open
samad13 wants to merge 2 commits intoOgstevyn:mainfrom
samad13:feature/Escrow-Status-Enum
Open

done#409
samad13 wants to merge 2 commits intoOgstevyn:mainfrom
samad13:feature/Escrow-Status-Enum

Conversation

@samad13
Copy link
Copy Markdown
Contributor

@samad13 samad13 commented Mar 28, 2026

PR Description: Define Escrow Status Enum

Description
This PR introduces the Status enum to the rent-escrow contract. This enum will be used to track the lifecycle of an escrow, moving from initial opening to funding, and finally to release or refund.

Changes

  • Added Status enum in [lib.rs]
  • escrow/src/lib.rs#L33-L42) with variants: Open, Funded, Released, and Refunded.
  • Mapped each variant to a u32 integer for storage efficiency within the Soroban environment.
  • Decorated the enum with #[contracttype] to enable its use in contract state and cross-contract calls.

Requirements

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 28, 2026

@samad13 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

@Ogstevyn
Copy link
Copy Markdown
Owner

@samad13 please resolve conflict

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.

Types: Define Escrow Status Enum

2 participants