Skip to content

Fix bounty arrow navigation at list boundaries#1616

Open
dhrubasumatary wants to merge 1 commit into
stakwork:masterfrom
dhrubasumatary:fix/bounty-arrow-zero-navigation
Open

Fix bounty arrow navigation at list boundaries#1616
dhrubasumatary wants to merge 1 commit into
stakwork:masterfrom
dhrubasumatary:fix/bounty-arrow-zero-navigation

Conversation

@dhrubasumatary
Copy link
Copy Markdown

Summary

  • Prevent public bounty L/R arrow navigation from routing to /bounty/0 when the API reports no next/previous bounty.
  • Reuse the existing "There are no more bounties to display!" toast behavior that workspace bounty navigation already had.
  • Add regression coverage for both public next and previous arrow paths.

Closes #333

Validation

  • npx.cmd prettier --config .prettierrc.json --check src/pages/tickets/TicketModalPage.tsx src/pages/tickets/__tests__/TicketModalPage.spec.tsx
  • npx.cmd eslint src/pages/tickets/TicketModalPage.tsx src/pages/tickets/__tests__/TicketModalPage.spec.tsx --max-warnings 100 (passes with existing warnings in these files)
  • npx.cmd jest src/pages/tickets/__tests__/TicketModalPage.spec.tsx --runInBand --no-coverage -t "shows a toast"
  • Full file run: all 27 assertions pass, but Jest exits nonzero because existing async backend calls log after test teardown when localhost:5002 is unavailable.

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 L/R arrows for org and homepage

1 participant