Skip to content

Fix/miner prs date status column#137

Open
plind-dm wants to merge 9 commits intoentrius:testfrom
plind-dm:fix/miner-prs-date-status-column
Open

Fix/miner prs date status column#137
plind-dm wants to merge 9 commits intoentrius:testfrom
plind-dm:fix/miner-prs-date-status-column

Conversation

@plind-dm
Copy link
Copy Markdown

@plind-dm plind-dm commented Apr 10, 2026

Summary

  • When the status filter is all or merged, the last column shows Date with the merge date (or Open/Closed for unmerged PRs)
  • When the status filter is open or closed, the column header switches to Status and displays the PR state label (Open/Merged/Closed) instead of a date

This makes the column contextually relevant — dates are most useful when viewing all or merged PRs, while status labels are more informative when already filtering by open or closed.

Changes

  • src/components/miners/MinerPRsTable.tsx — conditional header text and cell content based on statusFilter

Test plan

  • Navigate to a miner detail page → Pull Requests tab
  • With All filter: last column shows "Date" header with merge dates
  • With Merged filter: last column shows "Date" header with merge dates
  • With Open filter: last column shows "Status" header with "Open" labels
  • With Closed filter: last column shows "Status" header with "Closed" labels

Before
before
After
after

eureka0928 and others added 7 commits April 10, 2026 11:45
…igible

- Dashboard models: remove tier, currentTier, qualifiedUniqueReposCount,
  rawCredibility, credibilityScalar, repositoryUniquenessMultiplier,
  all bronze/silver/gold tier stats fields
- Add isEligible and credibility to MinerEvaluation
- Delete Predictions model and API
- Remove TierConfig from Configurations model and API
- Update leaderboard MinerStats type
Strip Bronze/Silver/Gold tier system across 51 files (~4950 lines removed).
Replace with flat isEligible boolean, update dynamic PR threshold from 500
to 300, revamp onboard Getting Started with 7-step inline setup guide
matching docs, remove Roadmap and FAQ tabs from onboard page.
🤖 Automated formatting by GitHub Actions
…ting

Replace dangerouslySetInnerHTML with ReactMarkdown + rehypeRaw for safe
rendering of GitHub-provided HTML in issue bodies and PR conversations.
Add missing CSS for lists, paragraphs, tables, and other elements that
MUI CssBaseline resets.

- Use ReactMarkdown with remarkGfm and rehypeRaw (same as ReadmeViewer)
- Add styles for ul/ol, li, p, table, hr, task checkboxes
- Fix pre/code nesting so inline code inside blocks renders correctly
- Left-align conversation panels to match GitHub layout
- Fix fallback text rendering (HTML em instead of markdown asterisks)
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…/merged

When statusFilter is open or closed, the last column header shows
"Status" with Open/Merged/Closed labels. When filter is all or merged,
it shows "Date" with the merge date.
@plind-dm plind-dm force-pushed the fix/miner-prs-date-status-column branch from c4852be to ae9ceb2 Compare April 10, 2026 19:40
@plind-dm
Copy link
Copy Markdown
Author

@LandynDev @anderdc How about adding new column "Status"?
Both Status and Date are shown in "Date" column now

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.

3 participants