Skip to content

feat(ranking): explain overall score calculation - #382

Merged
Willxup merged 2 commits into
mainfrom
worktree/ranking-score-explanation
Jul 29, 2026
Merged

feat(ranking): explain overall score calculation#382
Willxup merged 2 commits into
mainfrom
worktree/ranking-score-explanation

Conversation

@Willxup

@Willxup Willxup commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • forward the optional V2 score explanation from ranking-center through the Keeper API
  • show the center-provided text beside only the overall ranking title using the existing question-mark tooltip style
  • keep the tooltip readable on narrow screens and provide concise accessible labels in all supported languages
  • preserve compatibility when older centers omit the field

Validation

  • Go targeted ranking client and HTTP API tests
  • Keeper full local baseline for the cross-layer implementation
  • Keeper frontend baseline after review fixes: 103 test files and 958 tests, ESLint, TypeScript, and production build

Forward the optional ranking-center explanation through Keeper and show it beside the V2 overall ranking title with responsive, accessible tooltip behavior.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 139e08bbde

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread web/src/features/ranking/RankingPage.tsx
Treat a V2 explanation with omitted or null texts as absent so malformed center payloads cannot break the ranking page.
@Willxup
Willxup merged commit 56d30ac into main Jul 29, 2026
4 checks passed
@Willxup
Willxup deleted the worktree/ranking-score-explanation branch July 29, 2026 16:08
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.

1 participant