Skip to content

fix: add bounty list loading state#89

Open
tristanwork wants to merge 1 commit into
labmain:mainfrom
tristanwork:codex/bounty-18-loading-spinner
Open

fix: add bounty list loading state#89
tristanwork wants to merge 1 commit into
labmain:mainfrom
tristanwork:codex/bounty-18-loading-spinner

Conversation

@tristanwork

Copy link
Copy Markdown

Summary

  • add a bounty list loading controller that shows a spinner while the API request is pending
  • hide the spinner after both successful and failed requests
  • render a friendly error state when loading fails
  • add accessible loading spinner markup with status/aria-live semantics

Testing

  • npm test

Fixes #18

@tristanwork

Copy link
Copy Markdown
Author

Local verification passes on this branch:

npm test
# parser tests: 6 passed
# bounty list loading tests: 10 passed

The GitHub Actions check is currently action_required with no jobs started, so it appears to need maintainer approval to run for this fork PR.

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.

Add loading spinner to bounty list page

1 participant