Skip to content

✨ dashboard: Review Queue collapsed row shows a priority-split history sparkline - #10613

Merged
clubanderson merged 7 commits into
v5from
feat/review-queue-collapsed-sparkline-10610
Oct 5, 2026
Merged

clubanderson merged 7 commits into
v5from
feat/review-queue-collapsed-sparkline-10610

Conversation

@clubanderson

Copy link
Copy Markdown
Member

Closes #10610

Summary

  • Added priority-split Review Queue history to the review queue API response.
  • Replaced the collapsed Review Queue gauge with shared range controls, per-priority chips, delta, and a renderSparkline-backed priority history.
  • Added server/UI coverage and a changelog fragment.

Validation

  • git diff --check
  • gofmt -l src/pkg/dashboard/api_review_queue.go src/pkg/dashboard/api_review_queue_test.go src/pkg/dashboard/review_queue_view_9607_test.go
  • parsed inline dashboard JS with node --check

Note: per AGENTS.md, I did not run go test, go build, go vet, or citation guards locally.

Signed-off-by: Andy Anderson <andy@clubanderson.com>
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign danathar for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Oct 5, 2026
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Changelog: this PR changes code but carries no changelog entry
(no changelog.d/ fragment and no CHANGELOG.md edit).

If it is user-visible — a feature, a fix an operator would notice, a
security change, a migration, a deprecation, or anything breaking —
please add a fragment file changelog.d/<category>-<pr-or-slug>.md
containing exactly your entry (see changelog.d/README.md; the
category is one of added/changed/deprecated/fixed/security). Please
do not append to CHANGELOG.md's ## Unreleased directly — that
shared section is the merge-conflict generator fragments replaced
(#5675). If it is a refactor, a test-only change, or dependency
churn, no entry is needed and you can ignore this.

This is a reminder, not a gate; it never blocks a merge.

Generated by changelog-fragment-guard from the PR title (#5675).
Edit the wording freely, or replace this file with the no-changelog
label if the change is not user-visible.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: clubanderson <407614+clubanderson@users.noreply.github.com>
@kubestellar-prow kubestellar-prow Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 5, 2026
@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Open PR overlap check

Base branch: v5

Conflicts are checked with every PR merged onto the current tip of v5, so a PR that only needs a rebase is not reported as conflicting with its siblings.

PR Title Relation Files
#10655 ✨ Spek recheck cadence for continuous convergence same files src/pkg/dashboard/static/index.html
#10658 🐛 dashboard: Overview median-age KPI names its population and basis same files src/pkg/dashboard/static/index.html
#10614 ✨ dashboard: Audit Log collapsed row shows entry rate, last entry, today's count and sensitive-action chip same files src/pkg/dashboard/static/index.html

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Shared CI incident: the red checks on this PR (docker (hive), overlayfs-exec-guard, golangci-lint, relative links and anchors resolve, pkg/dashboard/pkg/hub test shards incl. TestAgentsCollapsedSummaryRendersPerAgentTiles / TestOpenAPISpecCoversEveryRegisteredRoute) are the v5 baseline breakage tracked in #10623, not this diff. Fix is carried by #10612; this PR is deferred until it lands, then needs a v5 merge/re-run.


🐝 Hive Agent: scanner | Instance: hosted-available-oke-11-placeholder-r05x | SHA: unknown

— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88

Signed-off-by: Andy Anderson <andy@clubanderson.com>
@clubanderson
clubanderson merged commit 5c7d262 into v5 Oct 5, 2026
62 of 63 checks passed
@clubanderson
clubanderson deleted the feat/review-queue-collapsed-sparkline-10610 branch October 5, 2026 18:15
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

We'd love to hear how your experience was: share feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ dashboard: Review Queue collapsed row — priority-split sparkline with history and range control

1 participant