rm-hull / github-pr-dashboard / 19775686663

56%

push

github

feat: Implement multi-state PR views and navigation (#112)

* feat: Implement multi-state PR views and navigation

This commit introduces proper routing and state management to display
Pull Requests and merged history using the same underlying list
component.

Key changes:
* Renamed `useOpenPullRequests` to `usePullRequests` to accept a
  state parameter (`open`, `closed`, `merged`).
* Updated `PullRequestsList` and list item actions to handle closed
  PR states correctly (e.g., disabling merge/rebase buttons).
* Added router navigation bar (`NavBar.tsx`) for Home, Issues,
  History, and Stats routes.
* Implemented History view using merged PRs.
* Updated Status icon component to display closed state.

* Update src/hooks/usePullRequests.ts

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update src/routes/github-pr-dashboard/history.tsx

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* refactor: Remove unnecessary useCallback in NavBar

Removes the unnecessary usage of `useCallback` around the
`isActive` utility function in the navigation bar component.

The function did not require memoization as it is defined and
used entirely within the render cycle.

* fix: Conditionally render Favicon component

The logic for updating the favicon alert count was previously rendered
unconditionally.

This change wraps both the `<Notifications />` and `<Favicon />`
components inside a conditional fragment dependent on
`enableNotifications`. If notifications are disabled, the favicon
alert logic is also disabled.

* refactor: Improve responsive display and PR actions

Updates several components to improve user experience:

*   The navigation bar heading shortens on smaller screens
    for better space utilization.
*   The `IgnoreButton` is now disabled for closed pull
    requests, as they cannot be ignored further.

---------

Co-authored-by: gemini-code-a... (continued)

9 of 11 branches covered (81.82%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

31 of 61 relevant lines covered (50.82%)

2.33 hits per line