Skip to content

feat: show queued transaction count in OfflineIndicator - #706

Merged
Jaydbrown merged 1 commit into
conduit-protocol:mainfrom
Team-work-007-1010:feat/offline-indicator-queued-count
Sep 28, 2026
Merged

Jaydbrown merged 1 commit into
conduit-protocol:mainfrom
Team-work-007-1010:feat/offline-indicator-queued-count

Conversation

@rhemy-arc

Copy link
Copy Markdown
Contributor

The offline banner now displays how many transactions are queued for synchronization instead of a generic message. The count updates in real time when transactions are queued or flushed.

  • Dispatch 'offline-transactions-changed' event from queueTransaction and flushQueuedTransactions
  • Listen for queue changes in OfflineIndicator and display count
  • Handle singular/plural forms ('1 transaction' vs 'N transactions')
  • Show default message when no transactions are queued
  • Add tests for queued count display

What does this PR do?

Type of change

  • Bug fix
  • New feature / page / component
  • Refactor
  • Style / design fix
  • Test coverage
  • Documentation
  • Dependency update

Related issue

Closes #

Closes #657
Closes #656
Closes #655
Closes #654

Changes

File Change

Checklist

  • npm run typecheck — no errors
  • npm run lint — no warnings
  • npm test — all tests pass
  • npm run build — production build succeeds
  • Tested in browser with Freighter on testnet
  • No hue-named Tailwind colour classes added (black/white/gray only)
  • Loading state handled for any new data-fetching UI
  • Error state handled and displayed inline (no alert())
  • 'use client' added only where genuinely needed
  • CHANGELOG.md updated under [Unreleased]

Screenshots

Before After

Notes for reviewers

The offline banner now displays how many transactions are queued for
synchronization instead of a generic message. The count updates in real
time when transactions are queued or flushed.

- Dispatch 'offline-transactions-changed' event from queueTransaction
  and flushQueuedTransactions
- Listen for queue changes in OfflineIndicator and display count
- Handle singular/plural forms ('1 transaction' vs 'N transactions')
- Show default message when no transactions are queued
- Add tests for queued count display
@drips-wave

drips-wave Bot commented Sep 28, 2026

Copy link
Copy Markdown

@rhemy-arc 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

2 participants