Skip to content

fix: show actual dates instead of Open/Closed in PR Date column (#1111)#1172

Closed
alpurkan17 wants to merge 2 commits into
entrius:mainfrom
alpurkan17:fix/1111-pr-date-column
Closed

fix: show actual dates instead of Open/Closed in PR Date column (#1111)#1172
alpurkan17 wants to merge 2 commits into
entrius:mainfrom
alpurkan17:fix/1111-pr-date-column

Conversation

@alpurkan17
Copy link
Copy Markdown

Summary

The PR table Date column rendered literal strings Open/Closed for non-merged PRs instead of actual dates, duplicating status info already shown in the Status column and tabs while hiding the real date.

Fix: use pr.closedAt || pr.prCreatedAt fallback instead of hardcoded Open/Closed strings in MinerPRsTable and search PullRequestsTab.

Validation

  • npm run build — no regressions
  • Merged PRs still show mergedAt
  • Closed PRs show closedAt (or prCreatedAt fallback)
  • Open PRs show prCreatedAt

Closes #1111

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 14, 2026
@alpurkan17
Copy link
Copy Markdown
Author

@anderdc @LandynDev ready for review

@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 15, 2026

PRs must target the test branch, not main — see CONTRIBUTING.md. Closing.

@anderdc anderdc closed this May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants