Skip to content

fix: Incorrect icon for Unknown sync status - #312

Open
keithchong wants to merge 1 commit into
redhat-developer:mainfrom
keithchong:bug-IncorrectIconForSyncStatus
Open

keithchong wants to merge 1 commit into
redhat-developer:mainfrom
keithchong:bug-IncorrectIconForSyncStatus

Conversation

@keithchong

Copy link
Copy Markdown
Collaborator

No Jira. Found this while testing. The current icon is spinning, which is wrong. It should just be an empty circle just like for a resource with no health status.

@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f38e0f63-9a82-40b2-a7fc-029b8d4e28be

@openshift-ci
openshift-ci Bot requested review from aali309 and wtam2018 November 2, 2026 17:22
@keithchong

Copy link
Copy Markdown
Collaborator Author

@aali309 could you please review this

@codecov-commenter

codecov-commenter commented Nov 2, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 14.16%. Comparing base (3bd1dab) to head (25669e4).
⚠️ Report is 75 commits behind head on main.

Files with missing lines Patch % Lines
...ponents/application/graph/icons/resource-icons.tsx 0.00% 2 Missing ⚠️
...itops/components/application/graph/graph-utils.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #312      +/-   ##
==========================================
+ Coverage   11.30%   14.16%   +2.85%     
==========================================
  Files         169      192      +23     
  Lines        6679     7535     +856     
  Branches     2235     2441     +206     
==========================================
+ Hits          755     1067     +312     
- Misses       5923     6467     +544     
  Partials        1        1              
Flag Coverage Δ
unit-tests 14.16% <0.00%> (+2.85%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Keith Chong <kykchong@redhat.com>
@keithchong
keithchong force-pushed the bug-IncorrectIconForSyncStatus branch from 4957801 to 25669e4 Compare November 2, 2026 17:26
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