Skip to content

Fix fee estimator#158

Open
T-kesh wants to merge 2 commits into
Sorokit:mainfrom
T-kesh:fix-fee-estimator
Open

Fix fee estimator#158
T-kesh wants to merge 2 commits into
Sorokit:mainfrom
T-kesh:fix-fee-estimator

Conversation

@T-kesh

@T-kesh T-kesh commented Jun 28, 2026

Copy link
Copy Markdown

feat: add robust FeeEstimator with concurrency guard, loading spinner, and retry button

  • Implements a concurrency guard using useRef to prevent overlapping load calls.
  • Adds safe client checks via hasClient().
  • Shows a non‑blocking loading spinner while preserving the last known fee.
  • Displays error messages with a retry button (accessible via ARIA label).
  • Polls fee estimates every 10 seconds with exhaustive dependency handling.
  • Includes live region for screen‑reader announcements.
  • Updated component tests to cover new UI and concurrency behavior.
    closes fix(FeeEstimator): concurrent load race condition, no loading skeleton, no retry button after error #13

@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@T-kesh 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

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.

fix(FeeEstimator): concurrent load race condition, no loading skeleton, no retry button after error

1 participant