Skip to content

[STUDIO-176] Confirm Before Deleting Databases and Tables - #503

Merged
dawsontoth merged 2 commits into
harper/fabricfrom
harper/STUDIO-176_deletion-confirmation
Jul 15, 2025
Merged

[STUDIO-176] Confirm Before Deleting Databases and Tables#503
dawsontoth merged 2 commits into
harper/fabricfrom
harper/STUDIO-176_deletion-confirmation

Conversation

@dawsontoth

@dawsontoth dawsontoth commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

I extracted the cluster deletion modal, made it generic, and then used it on the browse page.

https://harperdb.atlassian.net/browse/STUDIO-176

@dawsontoth
dawsontoth requested a review from a team as a code owner July 15, 2025 15:40
@dawsontoth
dawsontoth requested a review from BboyAkers July 15, 2025 15:40
@sonarqubecloud

Copy link
Copy Markdown

@BboyAkers

Copy link
Copy Markdown
Member

Really nice!!!!! 🔥

@dawsontoth
dawsontoth merged commit e1eeff8 into harper/fabric Jul 15, 2025
1 check passed
@dawsontoth
dawsontoth deleted the harper/STUDIO-176_deletion-confirmation branch July 15, 2025 18:37
dawsontoth added a commit that referenced this pull request Jul 24, 2026
Replace the mockup's placeholder SAMPLE data with a real useClusterUsage
query against the central-manager endpoint (HarperFast/central-manager#503).
Components and layout are unchanged — only the data source.

- getClusterUsage.ts: typed ClusterUsage response, query hook, and a
  usageSubtitle helper (plan name / cycle renewal / freshness).
- ClusterUsageCard: renders totals; hides while loading, on error, for
  self-hosted, or when there's no active plan/usage.
- Usage tab: loading spinner, self-hosted + no-usage empty states, and
  the metered / rate-limit / per-instance sections from live data.

The endpoint isn't on stage yet (CM #503 in review), so today the query
errors and the UI degrades gracefully (card hidden, tab shows the empty
state); it lights up once the endpoint deploys.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
dawsontoth added a commit that referenced this pull request Jul 28, 2026
Replace the mockup's placeholder SAMPLE data with a real useClusterUsage
query against the central-manager endpoint (HarperFast/central-manager#503).
Components and layout are unchanged — only the data source.

- getClusterUsage.ts: typed ClusterUsage response, query hook, and a
  usageSubtitle helper (plan name / cycle renewal / freshness).
- ClusterUsageCard: renders totals; hides while loading, on error, for
  self-hosted, or when there's no active plan/usage.
- Usage tab: loading spinner, self-hosted + no-usage empty states, and
  the metered / rate-limit / per-instance sections from live data.

The endpoint isn't on stage yet (CM #503 in review), so today the query
errors and the UI degrades gracefully (card hidden, tab shows the empty
state); it lights up once the endpoint deploys.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants