Skip to content

feat: expose queued deposits via public getter#271

Merged
liobrasil merged 5 commits intomainfrom
lionel/queued-deposits-getter-current-main
Mar 16, 2026
Merged

feat: expose queued deposits via public getter#271
liobrasil merged 5 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
@liobrasil liobrasil merged commit bc3d717 into main Mar 16, 2026
1 check passed
@liobrasil liobrasil deleted the lionel/queued-deposits-getter-current-main branch March 16, 2026 18:54
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.

2 participants