Skip to content

feat: expose queued deposits via public getter#271

Open
liobrasil wants to merge 4 commits intomainfrom
lionel/queued-deposits-getter-current-main
Open

feat: expose queued deposits via public getter#271
liobrasil wants to merge 4 commits intomainfrom
lionel/queued-deposits-getter-current-main

Conversation

@liobrasil
Copy link
Contributor

@liobrasil liobrasil commented Mar 13, 2026

Summary

  • add a public getQueuedDeposits(pid) getter on FlowALPv0.Pool
  • expose queued-deposit balances via a get_queued_deposits.cdc script and test helper
  • add the minimal FlowALPModels accessor needed for current main to read queued vault balances through the model abstraction
  • add integration coverage for queued deposit visibility and partial/full async drain

Notes

  • this supersedes the older FlowCreditMarket-based approach in Add queuedDepositAmounts mapping to track and query deposit amounts for scripts #71 for the current FlowALPv0 codebase
  • queueing behavior already existed; this PR makes queued balances publicly queryable on current main
  • queued balances are derived directly from the queued vaults, so no extra tracking map or sync migration is required in this implementation

Verification

  • flow test cadence/tests/queued_deposits_integration_test.cdc
  • flow test cadence/tests/deposit_capacity_test.cdc
  • flow test cadence/tests/async_update_position_test.cdc

@liobrasil liobrasil force-pushed the lionel/queued-deposits-getter-current-main branch from 54b925c to 6d04a3f Compare March 13, 2026 19:08
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.

1 participant