Repository navigation
feat(stream): useDebounce flush/cancel, bulk withdraw confirmation, timeline ledger captions, flow chart pulse - #716
Merged
Jaydbrown merged 1 commit intoOct 17, 2026
Conversation
…line ledgers, flow pulse
- useDebounce now returns [value, { flush, cancel }]; create-stream form
flushes a pending recipient check on submit instead of using stale state (conduit-protocol#672)
- BulkWithdrawButton shows a confirmation modal with the streams, total
claimable amount and minimum estimated network fees before withdrawing (conduit-protocol#669)
- StreamTimeline shows estimated ledger sequence captions under Start/End,
anchored to the latest ledger fetched on the stream page (conduit-protocol#668)
- StreamFlowChart adds an animated dashed pulse along the flow line for
active streams, disabled for prefers-reduced-motion (conduit-protocol#667)
Closes conduit-protocol#672, closes conduit-protocol#669, closes conduit-protocol#668, closes conduit-protocol#667
|
@Levi-Ojukwu 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! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Resolves four stream UI / hook issues.
[debouncedValue, { flush, cancel }](stablereferences). The create-stream page flushes the debounced recipient on submit,
so a pending validation can no longer leave stale state behind.
selected streams, total claimable balance and estimated network fees. No
wallet signing happens until the user confirms. Fees are a minimum
(base fee x configured multiplier x stream count); Soroban resource fees are
only known at signing time.
Ledger ≈ Nunder the Start and End events.The value is estimated at ~5s per ledger from the latest ledger fetched on the
stream page; captions are hidden if the RPC call fails. New
lib/ledger.tsand
getLatestLedgerSequence()inlib/soroban.ts.stroke-dashoffsetpulse along the flow linefor flowing streams only (not paused, cancelled or ended). Disabled under
prefers-reduced-motion.Breaking change
useDebouncenow returns a tuple instead of the bare value. The only caller(
app/create/page.tsx) is updated.Testing
no withdraw before confirm), ledger estimation, timeline captions, pulse
shown/hidden by stream state.
main(Tooltip, WithdrawButton,Navbar, etc.); no new ones.
app/settings/page.tsxhas a pre-existingsyntax error that
tscreports.closes #672
closes #669
closes #668
closes #667