Skip to content

Repo sidebar: always show activity chart and maintainers#1357

Merged
anderdc merged 1 commit into
testfrom
fix/repo-sidebar-chart-and-maintainers
Jul 12, 2026
Merged

Repo sidebar: always show activity chart and maintainers#1357
anderdc merged 1 commit into
testfrom
fix/repo-sidebar-chart-and-maintainers

Conversation

@e35ventura

Copy link
Copy Markdown
Collaborator

Problem

On repo detail pages (/miners/repository?name=...) the right sidebar was unstable:

  • The PR/issue activity chart only rendered while the Issues or Pull Requests tab was active, so it seemed to randomly appear and disappear as you moved between tabs (it never showed on the default Readme tab).
  • The maintainers section unmounted entirely whenever /repos/:repo/maintainers returned [], which it currently does whenever the backend's GitHub call is rate limited (see entrius/das-gittensor#87). Users saw the loading skeleton circles flash and then vanish.

Fix

  • Render the activity chart on every tab: issue mode on the Issues tab, PR mode everywhere else.
  • Fall back to the repo owner (avatar + GitHub profile link) when the API returns no maintainers, so the section is always present. Once das-gittensor#87 is deployed the API returns the real assignee list and the fallback only covers transient failures.

🤖 Generated with Claude Code

The PR/issue activity chart only rendered while the Issues or Pull
Requests tab was active, so it appeared and disappeared as you switched
tabs. Render it on every tab (issue mode on the Issues tab, PR mode
elsewhere).

The maintainers section unmounted entirely whenever the API returned an
empty list (which it currently does whenever the backend's GitHub call
is rate limited), leaving only a skeleton flash. Fall back to the repo
owner so the section is always present.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@anderdc
anderdc merged commit 1463f48 into test Jul 12, 2026
2 checks passed
@anderdc
anderdc deleted the fix/repo-sidebar-chart-and-maintainers branch July 12, 2026 23:34
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.

2 participants