Fixes #28735 Context Center Dashboard new UI by Rohit0301 · Pull Request #28652 · open-metadata/OpenMetadata

Rohit0301 · 2026-06-02T18:42:29Z

Describe your changes:

Screen.Recording.2026-06-03.at.10.27.45.PM.mov

Fixes 28735

I worked on ... because ...

Type of change:

High-level design:

N/A — small change.

Tests:

Use cases covered

Unit tests

Backend integration tests

Ingestion integration tests

Playwright (UI) tests

Manual testing performed

UI screen recording / screenshots:

Not applicable.

Checklist:

I have read the CONTRIBUTING document.
My PR title is Fixes <issue-number>: <short explanation>
My PR is linked to a GitHub issue via Fixes #<issue-number> above.
I have commented on my code, particularly in hard-to-understand areas.
For JSON Schema changes: I updated the migration scripts or explained why it is not needed.
For UI changes: I attached a screen recording and/or screenshots above.
I have added tests (unit / integration / Playwright as applicable) and listed them above.

Summary by Gitar

UI Components:
- Added ContextKnowledgePillarCard, AiActivitySection, and NeedsAttentionSection to the ContextCenterDashboardPage for a revamped UI.
Data Fetching Updates:
- Integrated listFolders and getListContextMemories to power new dashboard stats.
- Updated listContextFiles to support custom sorting/filtering and restricted data fetch limits using RECENT_DASHBOARD_ARTICLES_LIMIT and RECENT_DASHBOARD_DOCUMENTS_LIMIT.
Localization:
- Updated 19 language JSON files with new labels for AI activity, pillar stats, and status messaging.

_{This will update automatically on new commits.}

github-actions · 2026-06-02T19:08:19Z

Jest test Coverage

UI tests summary

Lines	Statements	Branches	Functions
	63.33% (67216/106129)	44.27% (36972/83510)	46.66% (11125/23842)

github-actions · 2026-06-02T21:59:03Z

🟡 Playwright Results — all passed (15 flaky)

✅ 4257 passed · ❌ 0 failed · 🟡 15 flaky · ⏭️ 88 skipped

Shard	Passed	Flaky	Skipped
✅ Shard 1	299	0	4
🟡 Shard 2	801	2	9
🟡 Shard 3	804	4	8
🟡 Shard 4	842	5	12
🟡 Shard 5	720	1	47
🟡 Shard 6	791	3	8

🟡 15 flaky test(s) (passed on retry)

Features/Glossary/GlossaryBulkOperations.spec.ts › should prevent dragging parent to its own child (shard 2, 1 retry)
Features/Glossary/GlossaryWorkflow.spec.ts › should display correct status badge color and icon (shard 2, 1 retry)
Features/KnowledgeCenterList.spec.ts › Knowledge Center List - Test bookmark functionality (shard 3, 1 retry)
Features/RTL.spec.ts › Verify Following widget functionality (shard 3, 1 retry)
Features/Tasks/TaskNavigation.spec.ts › navigating to /table/TASK-XXXXX should show 404 (invalid URL pattern) (shard 3, 1 retry)
Flow/ExploreAggregationCountsMatching.spec.ts › should verify left panel counts and tab search results for normal search (shard 3, 1 retry)
Flow/PersonaFlow.spec.ts › Set default persona for team should work properly (shard 4, 1 retry)
Pages/CustomProperties.spec.ts › Entity Reference (shard 4, 1 retry)
Pages/CustomProperties.spec.ts › Hyperlink (shard 4, 2 retries)
Pages/CustomProperties.spec.ts › Set enum custom property on column and verify in UI (shard 4, 2 retries)
Pages/DataContractsSemanticRules.spec.ts › Validate Description Rule Is_Set (shard 4, 1 retry)
Pages/EntityDataConsumer.spec.ts › Tier Add, Update and Remove (shard 5, 1 retry)
Pages/Lineage/LineageFilters.spec.ts › Verify lineage service type filter selection (shard 6, 1 retry)
Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
Pages/ServiceEntity.spec.ts › Inactive Announcement create & delete (shard 6, 1 retry)

📦 Download artifacts

How to debug locally

# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

chirag-madlani · 2026-06-05T11:22:48Z

+  return {
+    data: response.data.data ?? [],
+    paging: response.data.paging ?? {
+      total: (response.data.data ?? []).length,
+    },
+  };


Paging should always be there, can you confirm if response.data.data is correct?
Usually we have response.data & response.paging

this is correct response.data.data

gitar-bot · 2026-06-05T13:11:47Z

Code Review ✅ Approved 6 resolved / 6 findings

Implements the Context Center Dashboard UI revamp with updated components and data-fetching logic. Addressed findings related to data capping, type safety, and redundant mapping operations.

✅ 6 resolved

✅ Bug: Article/document stats show capped count instead of total

📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:327 📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:344
The stat prop for both the articles and documents pillar cards uses articles.length and documents.length respectively. Since RECENT_DASHBOARD_ARTICLES_LIMIT and RECENT_DASHBOARD_DOCUMENTS_LIMIT are both 3, the displayed count will always be at most "3" regardless of how many articles/documents actually exist.

The API (getListKnowledgePages) returns response.paging.total which gives the true total count. The same applies to listContextFiles. You should store and display the total from the paging response rather than the length of the fetched slice.

✅ Quality: Use typed interface instead of object for listContextFiles params

📄 openmetadata-ui/src/main/resources/ui/src/rest/assetAPI.ts:54
The params parameter in listContextFiles is typed as object, which provides no type safety and allows passing arbitrary properties. Since the API accepts specific sort parameters, a typed interface would be more appropriate and consistent with the codebase patterns (e.g., ListAssetsByFqnParams already exists in the same file).

✅ Quality: Redundant .map((page) => page) is a no-op

📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:149
The expression response.data.map((page: KnowledgePage) => page) creates a new array where each element maps to itself, which is functionally identical to just using the array directly (or spreading it if immutability is the concern).

✅ Quality: Hardcoded strings "0 this week", "0 processing", "0 cites today"

📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:332 📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:349 📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:365
The trend prop values passed to ContextKnowledgePillarCard are hardcoded English strings ("0 this week", "0 processing", "0 cites today") rather than using the useTranslation hook. This violates the i18n requirement and will not be translated for other locales.

✅ Edge Case: Removed null guard on listContextFiles data can crash .map

📄 openmetadata-ui/src/main/resources/ui/src/rest/assetAPI.ts:54-61 📄 openmetadata-ui/src/main/resources/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx:166-167
The old listContextFiles returned data: response.data.data ?? [], defensively defaulting to an empty array when the API omitted data. The refactored version returns response.data (a PagingResponse<ContextFile[]>) directly without any guard. Although the type declares data as required, at runtime an empty/204 or malformed response could leave data undefined, and the caller's response.data.map(contextFileToUploadedDocumentItem) (and response.data.length) at ContextCenterDashboardPage.tsx:166-167 would throw Cannot read properties of undefined. Consider defaulting data to [] either in the API helper or at the call site.

...and 1 more resolved from earlier reviews

Options

Display: compact → Showing less information.

Comment with these commands to change:

`Compact`
`gitar display:verbose`

_{Was this helpful? React with 👍 / 👎 | Gitar}

sonarqubecloud · 2026-06-05T13:35:38Z

Quality Gate passed for 'open-metadata-ui'

Issues
1440 New issues
0 Accepted issues

Measures
25 Security Hotspots
0.0% Coverage on New Code
3.5% Duplication on New Code

See analysis details on SonarQube Cloud

Rohit0301 added 2 commits June 3, 2026 00:01

Context Center Dashboard new UI

57405fe

lint fix

f5d5b0d

Rohit0301 self-assigned this Jun 2, 2026

Rohit0301 requested a review from a team as a code owner June 2, 2026 18:42

Rohit0301 added the safe to test Add this label to run secure Github workflows on PRs label Jun 2, 2026

Rohit0301 marked this pull request as draft June 2, 2026 18:42

gitar-bot Bot reviewed Jun 2, 2026

View reviewed changes

Comment thread ...ces/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx Outdated

gitar-bot Bot reviewed Jun 2, 2026

View reviewed changes

Comment thread openmetadata-ui/src/main/resources/ui/src/rest/assetAPI.ts Outdated

gitar-bot Bot reviewed Jun 2, 2026

View reviewed changes

Comment thread ...ces/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx Outdated

gitar-bot Bot reviewed Jun 2, 2026

View reviewed changes

Comment thread ...ces/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx

Rohit0301 temporarily deployed to test June 2, 2026 18:55 — with GitHub Actions Inactive

Rohit0301 had a problem deploying to test June 2, 2026 18:55 — with GitHub Actions Failure

Rohit0301 temporarily deployed to test June 2, 2026 18:55 — with GitHub Actions Inactive

minor fix

3906fb2

Rohit0301 marked this pull request as ready for review June 3, 2026 18:30

Merge branch 'main' into context-center-dashboard-new

a69a18a

Rohit0301 temporarily deployed to test June 3, 2026 18:45 — with GitHub Actions Inactive

Rohit0301 had a problem deploying to test June 3, 2026 18:45 — with GitHub Actions Failure

Rohit0301 temporarily deployed to test June 3, 2026 18:45 — with GitHub Actions Inactive

Rohit0301 and others added 2 commits June 4, 2026 13:12

playwright fix and addressed gitar comment

2561a90

Merge branch 'main' into context-center-dashboard-new

533429b

Rohit0301 had a problem deploying to test June 4, 2026 08:02 — with GitHub Actions Error

lint fix

432d740

Rohit0301 had a problem deploying to test June 5, 2026 11:22 — with GitHub Actions Error

chirag-madlani reviewed Jun 5, 2026

View reviewed changes

addressed PR comment

274e8ca

Rohit0301 had a problem deploying to test June 5, 2026 11:44 — with GitHub Actions Error

Merge branch 'main' into context-center-dashboard-new

2ce0a87

Rohit0301 had a problem deploying to test June 5, 2026 12:51 — with GitHub Actions Error

Rohit0301 added 2 commits June 5, 2026 18:35

remove duplicate interfaces

5918a39

lint fix

1fba63b

gitar-bot Bot reviewed Jun 5, 2026

View reviewed changes

Comment thread ...ces/ui/src/pages/ContextCenterPage/ContextCenterDashboardPage/ContextCenterDashboardPage.tsx

gitar-bot Bot reviewed Jun 5, 2026

View reviewed changes

Comment thread openmetadata-ui/src/main/resources/ui/src/rest/assetAPI.ts

Rohit0301 requested a deployment to test June 5, 2026 13:18 — with GitHub Actions In progress

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fixes #28735 Context Center Dashboard new UI#28652

Fixes #28735 Context Center Dashboard new UI#28652
Rohit0301 wants to merge 16 commits into
mainfrom
context-center-dashboard-new

Rohit0301 commented Jun 2, 2026 •

edited by gitar-bot Bot

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 2, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Jun 2, 2026 •

edited

Loading

Uh oh!

chirag-madlani Jun 5, 2026

Uh oh!

Rohit0301 Jun 5, 2026

Uh oh!

Rohit0301 Jun 5, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

gitar-bot Bot commented Jun 5, 2026

Uh oh!

sonarqubecloud Bot commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Rohit0301 commented Jun 2, 2026 • edited by gitar-bot Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Describe your changes:

Type of change:

High-level design:

Tests:

Use cases covered

Unit tests

Backend integration tests

Ingestion integration tests

Playwright (UI) tests

Manual testing performed

UI screen recording / screenshots:

Checklist:

Summary by Gitar

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Jest test Coverage

UI tests summary

Uh oh!

github-actions Bot commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🟡 Playwright Results — all passed (15 flaky)

Uh oh!

chirag-madlani Jun 5, 2026

Choose a reason for hiding this comment

Uh oh!

Rohit0301 Jun 5, 2026

Choose a reason for hiding this comment

Uh oh!

Rohit0301 Jun 5, 2026 • 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!

gitar-bot Bot commented Jun 5, 2026

Uh oh!

sonarqubecloud Bot commented Jun 5, 2026

Quality Gate passed for 'open-metadata-ui'

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Rohit0301 commented Jun 2, 2026 •

edited by gitar-bot Bot

Loading

github-actions Bot commented Jun 2, 2026 •

edited

Loading

github-actions Bot commented Jun 2, 2026 •

edited

Loading

Rohit0301 Jun 5, 2026 •

edited

Loading