Skip to content

Dashboard reservations: show only ACTIVE holds, drop history rows#89

Merged
LandynDev merged 1 commit into
testfrom
dashboard/reservations-active-only
May 12, 2026
Merged

Dashboard reservations: show only ACTIVE holds, drop history rows#89
LandynDev merged 1 commit into
testfrom
dashboard/reservations-active-only

Conversation

@e35ventura
Copy link
Copy Markdown
Collaborator

Summary

  • The Reservations card on the dashboard is a "miners currently held for an upcoming swap" view, but it was also listing INITIATED / EXPIRED / CANCELLED reservations — duplicating lifecycle state already visible in the Transactions card.
  • Filter the list to status === 'ACTIVE' so the section only reflects live holds. Update the empty-state copy to match ("No active reservations").

Test plan

  • Dashboard against https://api.all-ways.io shows only ACTIVE reservations in the Reservations card.
  • INITIATED reservations no longer appear; their corresponding swap row in Transactions is unchanged.
  • EXPIRED / CANCELLED reservations no longer appear in the card.
  • Empty state reads "No active reservations" when no active holds exist.
  • Source-address filter still works on the filtered list.

The Reservations card on the dashboard is a 'what's a miner currently
holding for you' view, but it was also surfacing INITIATED, EXPIRED, and
CANCELLED rows — duplicating lifecycle info already shown in the
Transactions card and crowding the live holds.

Filter the list to ACTIVE reservations only. Once a reservation becomes
INITIATED it has a swap, so it belongs in Transactions; expired and
cancelled rows are history with no actionable next step. The empty-state
copy is updated to match ('No active reservations').
@xiao-xiao-mao xiao-xiao-mao Bot added the enhancement New feature or request label May 12, 2026
Copy link
Copy Markdown
Collaborator

@LandynDev LandynDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay 😄

@LandynDev LandynDev merged commit 4a62697 into test May 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants