fix(scorecard): Scorecard aggregated metrics UI integration #2035

Eswaraiahsapram · 2026-01-12T10:54:42Z

Hey, I just made a Pull Request!

Fixes:

Description

Integrated Backend Aggregated metrics API with Frontend
In the existing entity scorecard component, we started using recharts pie chart
Added Missing permission for Aggregated metrics

Screen Recording

Screen.Recording.2026-01-12.at.4.41.35.PM.mov

✔️ Checklist

A changeset describing the change and affected packages. (more info)
Added or Updated documentation
Tests for new functionality and regression tests for bug fixes
Screenshots attached (for UI changes)

rhdh-gh-app · 2026-01-12T10:55:01Z

Changed Packages

Package Name	Package Path	Changeset Bump	Current Version
app	workspaces/scorecard/packages/app	none	`v0.0.0`
@red-hat-developer-hub/backstage-plugin-scorecard	workspaces/scorecard/plugins/scorecard	patch	`v2.2.0`

workspaces/scorecard/plugins/scorecard/src/components/Common/PermissionRequiredState.tsx

...scorecard/plugins/scorecard/src/components/Common/__tests__/PermissionRequiredState.test.tsx

dzemanov · 2026-01-14T14:43:12Z

Hmm, I noticed it just now when viewing the frontend code, why are we returning array from aggregations if it is always just 1 element in it with that concrete metric? cc @imykhno :D

dzemanov

I tested the PR with postgres and sqlite.

Screen.Recording.2026-01-14.at.15.14.22.mov

I was able to get Missing permission when all permissions missing or only for some scorecards with conditional permission, like here:

Translations worked fine for me.

When there is error in thresholds or in fetching metric, backend doesn't return the entity's result in response, this could be confusing to users if they own more entities but we return smaller total with no information that other entities had problems. When all entities have errors, backend returns [] and it leaves these empty spaces:

When there are no entities owned by user, backend also returns []. Maybe it would be good to return response also for 0 entities from backend, at least with metric metadata so it can be shown with 0 entities? If all entities have errors, it could help frontend if we return response with "status": "error".

workspaces/scorecard/plugins/scorecard/src/plugin.ts

...s/scorecard/plugins/scorecard/src/components/ScorecardHomepageSection/ResponsivePieChart.tsx

imykhno · 2026-01-15T07:48:39Z

I’ve tested the same scenarios locally as Dominika and observed the same results. However, when a user does not have access to the catalogue, the error shown in the screenshot below is displayed. Is this the expected behavior?

imykhno · 2026-01-15T07:58:45Z

One more point regarding SonarQube. I think it would be great if we could resolve or at least minimize the issues highlighted in the report.

ShiranHi · 2026-01-15T08:11:35Z

I’ve tested the same scenarios locally as Dominika and observed the same results. However, when a user does not have access to the catalogue, the error shown in the screenshot below is displayed. Is this the expected behavior?

My understanding is that the interface should dynamically hide the card for any user who lacks authorized access to the related catalog entities.

Co-authored-by: HusneShabbir <husneshabbir447@gmail.com>

Eswaraiahsapram · 2026-01-16T01:09:46Z

Thanks @christoph-jerolimov , @dzemanov , @imykhno for reviewing the PR. I addressed the review comments. PTAL

Screen.Recording.2026-01-16.at.6.33.54.AM.mov

christoph-jerolimov · 2026-01-16T08:55:35Z

I’ve tested the same scenarios locally as Dominika and observed the same results. However, when a user does not have access to the catalogue, the error shown in the screenshot below is displayed. Is this the expected behavior?

My understanding is that the interface should dynamically hide the card for any user who lacks authorized access to the related catalog entities.

Hi @ShiranHi, the homepage doesn't support this. We have a feature for the homepage to add RBAC checks to it. We will have that with the next release.

…log links) Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

christoph-jerolimov

Thanks @Eswaraiahsapram. This looks good to me 👍

christoph-jerolimov · 2026-01-16T08:58:11Z

workspaces/scorecard/packages/app/src/App.tsx

+    Component: ScorecardHomepageCard as ComponentType,
    config: {
-      layouts: {
-        xl: { w: 12, h: 6 },
-        lg: { w: 12, h: 6 },
-        md: { w: 12, h: 7 },
-        sm: { w: 12, h: 8 },
-        xs: { w: 12, h: 9 },
-        xxs: { w: 12, h: 10 },
+      id: 'scorecard-jira-homepage-section',
+      props: {
+        metricId: 'jira.open_issues',


Much better 👍 😄

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

sonarqubecloud · 2026-01-16T09:59:32Z

Quality Gate passed

Issues
9 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Eswaraiahsapram requested review from a team, christoph-jerolimov and its-mitesh-kumar as code owners January 12, 2026 10:54

Eswaraiahsapram force-pushed the scorecard-aggregated-metrics-ui-integration branch from 9d3db55 to e18a77f Compare January 12, 2026 11:02

Eswaraiahsapram requested review from ShiranHi, dzemanov, imykhno and rohitkrai03 January 12, 2026 11:59

Eswaraiahsapram force-pushed the scorecard-aggregated-metrics-ui-integration branch 2 times, most recently from 109e64e to 01b7ba1 Compare January 13, 2026 06:07

dzemanov requested changes Jan 13, 2026

View reviewed changes

workspaces/scorecard/plugins/scorecard/src/components/Common/PermissionRequiredState.tsx Outdated Show resolved Hide resolved

...scorecard/plugins/scorecard/src/components/Common/__tests__/PermissionRequiredState.test.tsx Outdated Show resolved Hide resolved

Eswaraiahsapram force-pushed the scorecard-aggregated-metrics-ui-integration branch from 01b7ba1 to e12e897 Compare January 13, 2026 13:22

Eswaraiahsapram requested a review from dzemanov January 13, 2026 13:23

dzemanov approved these changes Jan 14, 2026

View reviewed changes

dzemanov requested changes Jan 14, 2026

View reviewed changes

workspaces/scorecard/plugins/scorecard/src/plugin.ts Outdated Show resolved Hide resolved

imykhno reviewed Jan 14, 2026

View reviewed changes

...s/scorecard/plugins/scorecard/src/components/ScorecardHomepageSection/ResponsivePieChart.tsx Show resolved Hide resolved

imykhno and others added 8 commits January 16, 2026 05:47

feat(scorecard): implement endpoint to aggregate metrics for scorecard

c689d5d

fix(scorecard): integration UI with aggregated backend api

459b99b

add changeset

c4c7121

Remove unnecessary code changes

002af54

add api reports

c331082

fix lock file

dcdc01b

fix build api reports

fcd66f0

Co-authored-by: HusneShabbir <husneshabbir447@gmail.com>

fix e2e tests

9f4a4b4

Eswaraiahsapram added 3 commits January 16, 2026 05:47

fix e2e tests and sonarqube issues

cc97711

addressed review comments

4622857

unified error panel addressed review comments

c0f32eb

Eswaraiahsapram force-pushed the scorecard-aggregated-metrics-ui-integration branch from e12e897 to c0f32eb Compare January 16, 2026 01:02

fix api report

1991721

Eswaraiahsapram requested review from dzemanov and imykhno January 16, 2026 01:41

fix api report

72df96a

christoph-jerolimov added 4 commits January 16, 2026 10:04

chore(scorecard): simplify sidebar navigation (seperate home and cata…

69b1043

…log links) Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

chore(scorecard): add scorecards to the default homepage layout

734d88b

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

chore(scorecard): add scorecards card layout configuration

2638275

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

chore(scorecard): try customizable homepage card (doesn't work yet)

d025949

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

christoph-jerolimov approved these changes Jan 16, 2026

View reviewed changes

dzemanov approved these changes Jan 16, 2026

View reviewed changes

chore(scorecard): use catalog link to navigate to the catalog

c70fd6f

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

christoph-jerolimov merged commit 9dee966 into redhat-developer:main Jan 16, 2026
10 checks passed

fix(scorecard): Scorecard aggregated metrics UI integration #2035

fix(scorecard): Scorecard aggregated metrics UI integration #2035

Uh oh!

Conversation

Eswaraiahsapram commented Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Hey, I just made a Pull Request!

Description

Screen Recording

✔️ Checklist

Uh oh!

rhdh-gh-app bot commented Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changed Packages

Uh oh!

Uh oh!

Uh oh!

dzemanov commented Jan 14, 2026

Uh oh!

dzemanov left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

imykhno commented Jan 15, 2026

Uh oh!

imykhno commented Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ShiranHi commented Jan 15, 2026

Uh oh!

Eswaraiahsapram commented Jan 16, 2026

Uh oh!

christoph-jerolimov commented Jan 16, 2026

Uh oh!

christoph-jerolimov left a comment

Choose a reason for hiding this comment

Uh oh!

christoph-jerolimov Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Jan 16, 2026

Quality Gate passed

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Eswaraiahsapram commented Jan 12, 2026 •

edited

Loading

rhdh-gh-app bot commented Jan 12, 2026 •

edited

Loading

dzemanov left a comment •

edited

Loading

imykhno commented Jan 15, 2026 •

edited

Loading