Skip to content

Add active-round open positions drawer - #494

Open
Seunfunmi-319509 wants to merge 3 commits into
TevaLabs:mainfrom
Seunfunmi-319509:feature/411-open-positions-drawer
Open

Add active-round open positions drawer#494
Seunfunmi-319509 wants to merge 3 commits into
TevaLabs:mainfrom
Seunfunmi-319509:feature/411-open-positions-drawer

Conversation

@Seunfunmi-319509

Copy link
Copy Markdown
Contributor

Summary

Closes #411

Adds an open positions drawer to the active-round dashboard so connected users can quickly review predictions that are still unresolved.

What changed

  • Added an accessible Radix UI dialog/drawer with:
    • Empty state when there are no open predictions.
    • Position details for asset, direction, stake, potential payout, and exact target price.
    • Keyboard Escape handling, focus management, labelled title/description, and modal semantics.
    • Round links for each position and a link to the active round when available.
  • Added the dashboard entry point and open-position count badge.
  • Mapped prediction history data into open positions, including support for mock-friendly payloads and open-status variants.
  • Added focused component and dashboard tests for empty, populated, accessibility, close, navigation, and active-round states.
  • Synchronized npm/pnpm dependency locks and updated the PWA plugin so the existing npm ci CI workflow resolves successfully.
  • Stabilized the existing Vitest suite and fixed the pre-existing type/lint/test blockers found during CI verification.

Verification

  • npm test — passed
    • lint: 0 errors, 2 existing warnings
    • production build: passed
    • unit tests: 44 files, 521 tests passed
  • npm run test:a11y — passed, 4 tests
  • git diff --check — passed

Show connected users their unresolved predictions in an accessible Radix drawer, provide empty and populated states, and link each position back to its round. Synchronize dependency locks and stabilize the existing CI test suite so npm ci, lint, build, unit, and accessibility checks run successfully.

Generated with Codebuff 🤖
Co-Authored-By: Codebuff <noreply@codebuff.com>
@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

@Seunfunmi-319509 is attempting to deploy a commit to the josephchimebuka's projects Team on Vercel.

A member of the Team first needs to authorize it.

Seunfunmi-319509 and others added 2 commits August 26, 2026 12:15
Preserve the open positions drawer while incorporating the latest dashboard, wallet, event-log, asset-filter, and Soroban updates from upstream main. Keep the merged dependency manifests and tests consistent so the pull request can be evaluated against current main.

Generated with Codebuff 🤖
Co-Authored-By: Codebuff <noreply@codebuff.com>
Suppress the upstream effect lint rule at the existing data-loading boundaries so the merged branch passes the repository CI workflow while preserving the dashboard and open positions behavior.

Generated with Codebuff 🤖
Co-Authored-By: Codebuff <noreply@codebuff.com>
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.

Build my open positions drawer for active round

1 participant