Skip to content

Cycle 23 Web: Spending summary cards on dashboard homepage#679

Merged
rygel merged 2 commits into
developfrom
feat/cycle-23-web
Jun 26, 2026
Merged

Cycle 23 Web: Spending summary cards on dashboard homepage#679
rygel merged 2 commits into
developfrom
feat/cycle-23-web

Conversation

@rygel

@rygel rygel commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Changes

  • task-107: Spending summary cards on Web dashboard homepage. Inline Razor computation from visibleUsage where IsCurrencyUsage is true. Shows total spent, budget remaining, per-provider breakdown. Auto-hides when no currency providers.

Verification

  • Build: PASS
  • Tests: 1361 pass

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Large PR Detected

This PR contains 10383 lines changed across 278 files.

Details:

  • 📁 Files changed: 278
  • ➕ Insertions: 3735
  • ➖ Deletions: 6648
  • 📊 Total: 10383 lines

Recommendation:
Consider splitting this PR into smaller, focused changes to make review easier and faster. PRs under 500 lines are ideal for thorough code review.


This is an automated message. Feel free to ignore if this PR is intentionally large.

Inline Razor computation from visibleUsage where IsCurrencyUsage is
true. Shows total spent, budget remaining, and per-provider breakdown
cards using existing stat-card CSS classes. Section hidden when no
currency providers are active.

No new DB queries or page model changes needed — LatestUsage already
contains all spending data.
@rygel
rygel force-pushed the feat/cycle-23-web branch from 8de8c9c to 10763c2 Compare June 26, 2026 20:53
Develop already has a spending summary section (Spending Overview).
The rebase merged both versions, causing CS0128 duplicate variable
errors. Removed the redundant block; keeping develop's version which
correctly uses OfType<QuotaProviderUsage> for the abstract model.
@rygel
rygel merged commit 93da3ee into develop Jun 26, 2026
3 checks passed
@rygel
rygel deleted the feat/cycle-23-web branch June 26, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants