feat: add admin search analytics page with stubbed backend (#326) - #440
feat: add admin search analytics page with stubbed backend (#326)#440ojuotimi932 wants to merge 3 commits into
Conversation
…ge#326) Add the search analytics dashboard page showing query volume trends, click-through rates, top queries table, and zero-result content gap signals. Backend action is stubbed with tracked:false placeholders until analytics endpoints ship. Includes service contract and page state tests. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
|
@ojuotimi932 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
Someone is attempting to deploy a commit to the Deen Bridge Team on Vercel. A member of the Team first needs to authorize it. |
|
Warning Review limit reachedNext included review available in 3 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe change adds a client-side admin search analytics page. A stubbed action returns a typed analytics snapshot with date ranges, metrics, query lists, and weekly trends. Tests cover the service contract and page states. ChangesSearch analytics
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: 🔵 Low · up to The new admin analytics page currently shows only placeholder data, so immediate user or production impact is limited; however, the route and analytics service lack an enforced administrator authorization boundary, so owner follow-up is required before real analytics are connected. Sequence Diagram(s)sequenceDiagram
participant Admin
participant SearchAnalyticsPage
participant fetchSearchAnalytics
participant AnalyticsView
Admin->>SearchAnalyticsPage: choose time range
SearchAnalyticsPage->>fetchSearchAnalytics: request analytics date range
fetchSearchAnalytics-->>SearchAnalyticsPage: return analytics snapshot
SearchAnalyticsPage->>AnalyticsView: render metrics, charts, and tables
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Linked Issues checkExplanation The PR implements the search analytics page, top-query and zero-result tables, trend visualizations, date-range selection, and the aggregation contract. The stubbed backend supports the contract and intentional tracked-state placeholders described in the PR objectives. ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Fix duplicate/corrupted JSX structures from bad merges in: - audit-logs: removed duplicate table headers, loading states, and pagination buttons - reconciliation: removed duplicate button block and duplicate tx.txHash section - reports: removed duplicate table headers, loading states, and dropdown menus Also fix missing semicolon in lib/admin/messages/common.js (unescaped quotes). 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
- Fix lighthouserc.json: change urlList → url for lighthouse-ci - Fix a11y label/control associations in audit-logs, reconciliation, GlobalTransactionExplorer (add htmlFor + id) - Add role/tabIndex/onKeyDown to clickable div in reports page - Regenerate package-lock.json to resolve dependency mismatches 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
|
Strict review blocker: |
|
Strict review blocker: required CI checks are failing. Please fix them before requesting merge. |
|
Strict review blocker: and are failing. Please fix the failing checks before requesting merge. |
|
@ojuotimi932 this PR has merge conflicts with the |
2 participants
Closes #326
Add the search analytics dashboard page showing query volume trends, click-through rates, top queries table, and zero-result content gap signals. Backend action is stubbed with tracked:false placeholders until analytics endpoints ship. Includes service contract and page state tests.
🤖 Generated with Codebuff
Summary by CodeRabbit
New Features
Tests