Skip to content

fix: scope faucet pending state per token#176

Open
9904099 wants to merge 1 commit into
SO4-Markets:mainfrom
9904099:fix/faucet-per-token-pending
Open

fix: scope faucet pending state per token#176
9904099 wants to merge 1 commit into
SO4-Markets:mainfrom
9904099:fix/faucet-per-token-pending

Conversation

@9904099

@9904099 9904099 commented Jun 20, 2026

Copy link
Copy Markdown

Summary

  • Split faucet claim pending state into per-token pending state and separate bulk pending state.
  • Show the Claiming... spinner only on the individual token card that is currently claiming.
  • Keep the bulk Claim Test Tokens button pending state independent from individual card claims.
  • Guard against rapid double-submits by tracking pending tokens synchronously while React state updates.

Closes #175

Validation

  • bun install --frozen-lockfile
  • bun run lint src/features/faucet/hooks/useClaim.tsx src/features/faucet/components/faucet-page.tsx
  • bun run build

Note: full bun run --filter web typecheck currently reports pre-existing unrelated errors outside the faucet files.

@vercel

vercel Bot commented Jun 20, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the Ijai's projects Team on Vercel.

A member of the Team first needs to authorize it.

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 #2 — Faucet: give each token card its own claim state

1 participant