Skip to content

fix: show analyzing status instead of 0 during analysis - #382

Open
itxaiohanglover wants to merge 1 commit into
oss-compass:mainfrom
itxaiohanglover:fix/depth-insight-analyzing-status
Open

fix: show analyzing status instead of 0 during analysis#382
itxaiohanglover wants to merge 1 commit into
oss-compass:mainfrom
itxaiohanglover:fix/depth-insight-analyzing-status

Conversation

@itxaiohanglover

Copy link
Copy Markdown

Summary

Fixes #285

When a project's analysis is still in progress, the depth insight page (深度洞察) shows "0" for all metrics. This happens because the data API returns 500 when the JSON data file doesn't exist yet, and the client silently falls back to empty arrays.

Changes

  • Add isError check to both region queries in GovernanceTabPanel
  • When both queries fail (data unavailable = analysis in progress), render the existing Analyzing component instead of the dashboard with zero values

Test plan

  • Open a project page where analysis is in progress
  • Verify "报告分析中" is displayed instead of "0" values
  • Open a project page with available data — verify the dashboard renders normally

@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

@itxaiohanglover is attempting to deploy a commit to the codersett's projects Team on Vercel.

A member of the Team first needs to authorize it.

When a project's analysis is still in progress, the depth insight page
shows "0" for all metrics because the data API returns 500 and the
client falls back to empty arrays. This adds an early check: if both
region queries fail, render the Analyzing component instead of the
dashboard with zero values.

Fixes oss-compass#285

Signed-off-by: itxaiohanglover <1531137510@qq.com>
@itxaiohanglover
itxaiohanglover force-pushed the fix/depth-insight-analyzing-status branch from 844db95 to c0559b3 Compare July 7, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant