Agents Builders

Add NFL hub link to admin dashboard

Archived
task-fb455eb7207c

Created

Jun 20, 16:36

Started

Jun 20, 16:36

Completed

Jun 21, 04:09

DevOps handoff

Type

Feature

Shape

ui-only

Worktree Slug

admin-dashboard-nfl-hub-link

Repositories

mcritchie-studio

Release Train

Branch

feat/admin-dashboard-nfl-hub-link

QA URL

Production URL

ui

Acceptance Criteria

  • Admin dashboard includes a clear link to the NFL hub.
  • The link points at the existing NFL hub route.
  • Existing dashboard content and layout remain intact.

Expected Test Plan

  • [component] Admin dashboard controller/render test verifies the NFL hub quick link.
  • [manual] Review http://localhost:3001/admin/dashboard and follow the NFL Hub shortcut.

Checks Run

  • [review] Avi QA SHIP — route nfl_hub_path→/nfl resolves to real NflController#index+view; grid xl:grid-cols-7 clean for 7 links; integration test asserts link for admin; all 3 acceptance criteria met; CI 4/4 green, mergeable

Stage Timeline

Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. means the agent didn't report that metric.

No stage changes recorded yet.

Conversation

QA review feedback, agent handoffs, and follow-up notes for this task.

Handoff 7 days ago

Added the NFL Hub quick link to /admin/dashboard using the existing quick-link partial, adjusted the shortcut grid for six links, and added controller/render coverage for nfl_hub_path. PR: https://github.com/amcritchie/mcritchie-studio/pull/67. Local proof: http://localhost:3001/admin/dashboard. Checks: [component] bin/rails test test/controllers/admin/dashboard_controller_test.rb; [gate] bin/dor-check task-fb455eb7207c.

QA Feedback 7 days ago

DevOps review: change good, gates green. Before merge: (1) mark PR #67 ready (still draft); (2) rebase onto main after #66 lands — same insertion point in show.html.erb. Keep both cards + the cols-6 grid upgrade.

QA Feedback 7 days ago

Review: the change itself is fine, but blocked for rework — (1) PR #67 is still a DRAFT; mark it ready for review. (2) Rebase onto current main (it's behind + CI is stale, pre-v77 deploy). (3) It conflicts with #66 (same lines of admin/dashboard/show.html.erb); #66 is approved first, so rebase #67 on top of it. Re-submit when green + ready.

Handoff 6 days ago

Cleared the PR #67 blocker: rebased feat/admin-dashboard-nfl-hub-link onto current main (6cfdfcf), preserved the shipped Tasks quick link from #66, resolved the dashboard/test conflicts, force-pushed with lease, and marked the PR ready for review. Verification: [component] bin/rails test test/controllers/admin/dashboard_controller_test.rb passed; [gate] bin/dor-check task-fb455eb7207c passed; [ci] gh pr checks 67 passed lint, scan_js, scan_ruby, and test. Local review URL is healthy again at http://localhost:3001/admin/dashboard after applying the worktree dev DB migrations.

Sealed-bid sizing

Edit →

Alex (PM)

Avi (PO)

Dev

Actual