cheapnud/CheapClerk
Watch
1
0
Fork
You've already forked CheapClerk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use archive totals on the dashboard and show basic metrics #17

Merged
cheapnud merged 1 commit from b/dashboard-stats into master 2026-08-18 00:09:26 +00:00
Conversation 1 Commits 1 Files changed 7 +89 -9
cheapnud commented 2026-08-18 00:06:31 +00:00
Owner
Copy link

Dashboard counts come from the statistics endpoint now instead of the first page of the document list, and a small metrics strip shows the file-type mix, OCR character volume and document-type count.

Dashboard counts come from the statistics endpoint now instead of the first page of the document list, and a small metrics strip shows the file-type mix, OCR character volume and document-type count.
Use archive totals on the dashboard and show basic metrics
All checks were successful
Claude PR Review / AI Code Review (pull_request) Successful in 24s
Details
Build & Test / Build and Test (pull_request) Successful in 2m29s
Details
15040758c5
forgejo-actions commented 2026-08-18 00:06:55 +00:00
Copy link

🤖 Claude AI Code Review

  1. Summary
    Dashboard now sources counts from the statistics endpoint and adds a small metrics strip. Clean, well-scoped change.

  2. Code Quality ⭐⭐⭐⭐☆
    Good use of parallel task fetching and sensible null-fallbacks for counts.

  3. Issues

🟡 Home.razor — _documentCount fallback uses _recentDocuments.Count (capped at 10) if statistics is null, which is exactly the paginated-page-size bug this PR aims to fix. Minor, but if the stats call fails the count silently reverts to being wrong. Acceptable as a degraded fallback, but consider showing an unknown/loading state instead.

🟢 TargetFramework is net11.0 — verify this is intentional (net11.0 is not a released TFM at time of writing). Likely pre-existing and out of scope.

Note: the UserSecretsId GUID is not a secret (it only maps to local secret storage), so no exposure concern.

  1. Verdict: APPROVE

The fallback behavior is a reasonable degraded case and everything else is correct and well-scoped.

📊 Tokens: 5021 input + 389 output | 💰 Cost: ~$0.0348
Generated by Claude Opus 4.8 via Gitea Actions

## 🤖 Claude AI Code Review 1. **Summary** Dashboard now sources counts from the statistics endpoint and adds a small metrics strip. Clean, well-scoped change. 2. **Code Quality** ⭐⭐⭐⭐☆ Good use of parallel task fetching and sensible null-fallbacks for counts. 3. **Issues** 🟡 Home.razor — `_documentCount` fallback uses `_recentDocuments.Count` (capped at 10) if statistics is null, which is exactly the paginated-page-size bug this PR aims to fix. Minor, but if the stats call fails the count silently reverts to being wrong. Acceptable as a degraded fallback, but consider showing an unknown/loading state instead. 🟢 TargetFramework is `net11.0` — verify this is intentional (net11.0 is not a released TFM at time of writing). Likely pre-existing and out of scope. Note: the `UserSecretsId` GUID is not a secret (it only maps to local secret storage), so no exposure concern. 4. **Verdict**: **APPROVE** The fallback behavior is a reasonable degraded case and everything else is correct and well-scoped. --- 📊 **Tokens**: 5021 input + 389 output | 💰 **Cost**: ~$0.0348 *Generated by Claude Opus 4.8 via Gitea Actions*
cheapnud merged commit 0c70bd2430 into master 2026-08-18 00:09:26 +00:00
cheapnud deleted branch b/dashboard-stats 2026-08-18 00:09:27 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
cheapnud/CheapClerk!17
No description provided.