Fix/leaderboard sidebar duplicate key#124
Open
tmimmanuel wants to merge 10 commits intoentrius:testfrom
Open
Conversation
anderdc
reviewed
Apr 9, 2026
| (miner, i) => ( | ||
| <LeaderboardRow | ||
| key={miner.hotkey} | ||
| key={miner.githubId || miner.hotkey} |
Collaborator
There was a problem hiding this comment.
hmm I'd prefer the keys to only have 1 value, would it be better to just use githubId in both the cases you updated?
Author
|
@anderdc I fixed. |
…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>
tmimmanuel
force-pushed
the
fix/leaderboard-sidebar-duplicate-key
branch
from
7aa3343 to
347b793
Compare
Author
|
@anderdc Please review my PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
miner.hotkey(can be shared/N/A) tominer.githubId(unique per row)Type of Change
Screenshots
video.mp4