Skip to content

Implement typed storage helpers#100

Open
TUPM96 wants to merge 1 commit into
JSE-ORG:mainfrom
TUPM96:codex/storage-helpers-11
Open

Implement typed storage helpers#100
TUPM96 wants to merge 1 commit into
JSE-ORG:mainfrom
TUPM96:codex/storage-helpers-11

Conversation

@TUPM96

@TUPM96 TUPM96 commented May 25, 2026

Copy link
Copy Markdown

Closes #11

Summary

  • Move storage keys into a typed storage module with helper functions for escrows, vendor/buyer indexes, escrow counter, admin address, and fee config.
  • Refactor escrow flows to use the storage helpers instead of direct business-logic storage access.
  • Add vendor/buyer escrow index view methods and tests, including updated Soroban snapshots.

Validation

  • cargo fmt --check
  • cargo test (14 passed; existing deprecation warning remains for register_stellar_asset_contract in tests)
  • cargo build --target wasm32v1-none --release
  • git diff --check

Copilot AI review requested due to automatic review settings May 25, 2026 10:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@JSE19

JSE19 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

👋 main has been fixed — \ und_escrow\ and
aise_dispute\ are restored and clippy is clean. Could you rebase your branch on the latest main so CI re-runs? \git rebase main\ then force-push.

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.

Issue #11: Implement storage key constants and helpers in storage.rs

3 participants