Auto: feature/admin-dashboard-cleanup #30

Closed

claude-bot wants to merge 2 commits from feature/admin-dashboard-cleanup into main

pull from: feature/admin-dashboard-cleanup

merge into: bittabola:main

bittabola:main

Conversation 1 Commits 2 Files changed 1 +108 -92

claude-bot commented 2026-03-03 21:15:30 -07:00

Collaborator

Copy link

Automated PR for branch feature/admin-dashboard-cleanup.

Automated PR for branch `feature/admin-dashboard-cleanup`.

claude-bot added 2 commits 2026-03-03 21:15:31 -07:00

Clean up admin dashboard: remove stale metrics, apply shadcn-style CSS ... a1917c2dd5

Remove Input/Output Tokens and Premium/Free API Cost cards from Overview
that are no longer useful for a single-service app. Replace inline CSS
with shadcn-inspired design system using CSS variables, light header with
tab navigation, bordered cards, muted table headers, and pill badges.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Remove unnecessary header-top wrapper div ... 99bf6438e7

Address review suggestion: simplify header markup since h1 has no
sibling elements that require a flex container.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude-bot commented 2026-03-03 21:16:31 -07:00

Author

Collaborator

Copy link

Approved

This is a clean UI refactoring of the admin dashboard: hardcoded CSS colors are replaced with CSS custom properties (shadcn-style design tokens), and four granular stat cards are removed in favor of keeping only the totals. No logic, security, or performance issues found.

Suggestions

• admin_dashboard.py: Three stats keys (total_input_tokens, total_output_tokens, free_cost_usd) are still being computed via SQL and stored in the stats dict but are no longer displayed or used anywhere. Consider removing them from get_overview_stats() to avoid unnecessary DB computation and reduce confusion for future maintainers. (premium_cost_usd is still needed for the premium_profit_loss calculation.)
• admin_dashboard.py: Minor: the CSS variable --ring (#94a3b8) is declared in :root but never referenced in any rule. Consider removing it to keep the stylesheet minimal, or add a comment noting it's reserved for focus-ring styling to be added later.

## Approved This is a clean UI refactoring of the admin dashboard: hardcoded CSS colors are replaced with CSS custom properties (shadcn-style design tokens), and four granular stat cards are removed in favor of keeping only the totals. No logic, security, or performance issues found. ### Suggestions - **admin_dashboard.py**: Three stats keys (total_input_tokens, total_output_tokens, free_cost_usd) are still being computed via SQL and stored in the stats dict but are no longer displayed or used anywhere. Consider removing them from get_overview_stats() to avoid unnecessary DB computation and reduce confusion for future maintainers. (premium_cost_usd is still needed for the premium_profit_loss calculation.) - **admin_dashboard.py**: Minor: the CSS variable --ring (#94a3b8) is declared in :root but never referenced in any rule. Consider removing it to keep the stylesheet minimal, or add a comment noting it's reserved for focus-ring styling to be added later.

bittabola closed this pull request 2026-03-03 22:17:08 -07:00

Some checks failed

Hide all checks

Auto PR Review / review (push) Failing after 1m41s

Details

Pull request closed

Please reopen this pull request to perform a merge.

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

1 participant

Notifications

Subscribe

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

bittabola/tarjimon!30

No description provided.