🐛 Fix 7 data refresh verification bugs#169
Conversation
1. Namespace dedup: Use deduplicatedClusters in NamespaceManager to
avoid fetching from duplicate long context names
2. Deployment Progress spinner: Only show loading spinner when no
data at all, not when demo/initial data is available
3. Deployments stats: Show "0" instead of "-" when no deployments
by using hasData={!isLoading} instead of hasData={count > 0}
4. Dashboard Namespaces stat: Return actual namespace count with
"namespaces" sublabel instead of node count with "nodes"
5. "Never refreshed" badge: Set lastRefresh in CacheStore when no
IndexedDB data exists so cards don't show stale badge
6. Deployment "unknown" cluster: Enrich deployment objects with
cluster name in all three fetch paths (agent, kubectl, REST API)
7. Dead code: Remove unused ResourceDetailModal.tsx (450 lines,
never imported anywhere)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Andrew Anderson <andy@clubanderson.com>
Add Phase 5 section documenting all 13 HTTP + WebSocket agent endpoints with auth requirements and test protocol. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Signed-off-by: Andrew Anderson <andy@clubanderson.com>
kubestellar-prow
Bot
added
the
dco-signoff: yes
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for kubestellarklaudeconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Welcome to KubeStellar! 🚀 Thank you for submitting this Pull Request. Before your PR can be merged, please ensure: ✅ DCO Sign-off - All commits must be signed off with ✅ PR Title - Must start with an emoji: ✨ (feature), 🐛 (bug fix), 📖 (docs), 🌱 (infra/tests), Getting Started with KubeStellar: Contributor Resources:
🌟 Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback. Whether you're using KubeStellar yourself or know someone who could benefit from multi-cluster Kubernetes: 📋 Take our Multi-Cluster Survey - Share your use cases and help shape our direction! A maintainer will review your PR soon. Feel free to ask questions in the comments or on Slack! |
kubestellar-prow
Bot
added
the
size/XL
|
🎉 Thank you for your contribution! Your PR has been successfully merged. 🌟 Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback - nothing else. Whether you're using KubeStellar yourself or know organizations that could benefit from multi-cluster Kubernetes, we need your help: 📋 Take our Multi-Cluster Survey - Share your use cases and help shape our direction! 🗣️ Spread the word - Tell colleagues, write blog posts, present at meetups 💬 Share feedback on Slack #kubestellar-dev Every adopter story helps us prioritize what matters most. Thank you for being part of the KubeStellar community! |
❌ Playwright Tests Failed📊 View Full ReportDownload the To view the report locally: # Download and extract playwright-report.zip
npx playwright show-report path/to/playwright-report |
* 🐛 Fix 7 data refresh verification bugs
1. Namespace dedup: Use deduplicatedClusters in NamespaceManager to
avoid fetching from duplicate long context names
2. Deployment Progress spinner: Only show loading spinner when no
data at all, not when demo/initial data is available
3. Deployments stats: Show "0" instead of "-" when no deployments
by using hasData={!isLoading} instead of hasData={count > 0}
4. Dashboard Namespaces stat: Return actual namespace count with
"namespaces" sublabel instead of node count with "nodes"
5. "Never refreshed" badge: Set lastRefresh in CacheStore when no
IndexedDB data exists so cards don't show stale badge
6. Deployment "unknown" cluster: Enrich deployment objects with
cluster name in all three fetch paths (agent, kubectl, REST API)
7. Dead code: Remove unused ResourceDetailModal.tsx (450 lines,
never imported anywhere)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Andrew Anderson <andy@clubanderson.com>
* 📖 Add agent API endpoint testing to consistency.md
Add Phase 5 section documenting all 13 HTTP + WebSocket agent
endpoints with auth requirements and test protocol.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Andrew Anderson <andy@clubanderson.com>
---------
Signed-off-by: Andrew Anderson <andy@clubanderson.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
1 participant
Summary
Fixes 7 bugs discovered during comprehensive data refresh verification testing of the KubeStellar Console UI:
clusters(with duplicates) instead ofdeduplicatedClusters, causing failed API calls to long context namesclusterfield, showing "unknown" in UIResourceDetailModal.tsxthat was never imported anywhereAlso adds agent API endpoint testing documentation to
consistency.md.Changes
web/src/components/namespaces/NamespaceManager.tsxdeduplicatedClustersfor API callsweb/src/components/cards/DeploymentProgress.tsxweb/src/components/deployments/Deployments.tsxhasDatato use!isLoadingweb/src/components/dashboard/Dashboard.tsxweb/src/lib/cache/index.tslastRefreshwhen no IDB entry existsweb/src/hooks/useCachedData.tsweb/src/hooks/useMCP.tsweb/src/components/dashboard/ResourceDetailModal.tsxconsistency.mdTest plan
npm run buildpasses cleanly/namespacespage shows only deduplicated cluster names in errors/deploymentsstats show "0" instead of "-"🤖 Generated with Claude Code