Skip to content

Backport #29066 #29063 #29069 services lineage and data insight lazy-loading to 1.13 - #31048

Open
shah-harshit wants to merge 1 commit into
ui/direct-1.13-seq29-29068-explore-search-alerts-lazyfrom
ui/direct-1.13-seq30-32-services-lineage-data-insight-lazy
Open

Backport #29066 #29063 #29069 services lineage and data insight lazy-loading to 1.13#31048
shah-harshit wants to merge 1 commit into
ui/direct-1.13-seq29-29068-explore-search-alerts-lazyfrom
ui/direct-1.13-seq30-32-services-lineage-data-insight-lazy

Conversation

@shah-harshit

Copy link
Copy Markdown
Contributor

…1.13 (#30136)

* feat(ui): update Services & Ingestion components for lazy-load utility pattern (#29066)

* feat(ui): update Services & Ingestion components for lazy-load utility pattern

Updates Services and Ingestion components to import from extracted pure utility modules. Updates ServiceUtils, IngestionUtils, SchedularUtils imports. Updates service-related pages for consistent utility usage.

Ref: open-metadata/openmetadata-collate#4230

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* minor fix

* fix checkstyle and tests

* fix build

* fix checkstyle

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(ui): update Entity & Lineage components for lazy-load utility pattern (#29063)

* feat(ui): update Entity & Lineage components for lazy-load utility pattern

Updates Entity and Lineage components to import from extracted pure utility modules. Updates LineageUtils, EntityVersionUtils, CanvasUtils imports. Updates EntityBreadcrumbUtils and EntitySummaryPanelUtils references.

Ref: open-metadata/openmetadata-collate#4230

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix failing checks

* fix build

* fix build, checkstyle and tests

* fix e2e tests

* fix checkstyle

* fix failing test

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(ui): update Dashboard, Pipeline, Chart & ML components for lazy-load utility pattern (#29069)

* feat(ui): update Dashboard, Pipeline, Chart & ML components for lazy-load utility pattern

Updates Dashboard, Pipeline, Chart, DataInsight and ML components to import from extracted pure utility modules. Updates DataInsightUtils, MetricUtils imports. Updates APIEndpoint and SearchIndex components for consistent utility usage.

Ref: open-metadata/openmetadata-collate#4230

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix test, build and checkstyle

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(ui): resolve group15 backport issues

* fix(ui): update notification feed test mock

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
(cherry picked from commit 8c9a856)
@shah-harshit
shah-harshit requested a review from a team as a code owner August 5, 2026 13:33
@shah-harshit shah-harshit added UI UI specific issues safe to test Add this label to run secure Github workflows on PRs skip-pr-checks Bypass PR metadata validation check labels Aug 5, 2026
@shah-harshit shah-harshit self-assigned this Aug 5, 2026
@greptile-apps

greptile-apps Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review (115 files, 100 file limit).

Bypass the limit by tagging @greptile-apps to review.

@gitar-bot

gitar-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Backports services, lineage, and data insight lazy-loading updates to branch 1.13 via stacked cherry-pick. No issues found.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar | Powered by Gitar — free for open source

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

❌ UI Checkstyle Failed

❌ ESLint + Prettier + Organise Imports (src)

One or more source files have linting or formatting issues.

❌ Tailwind Audit

Hardcoded Tailwind values found. Use a design-system utility (run yarn tw-audit:report for the token each value maps to).

❌ Antd + Less Deprecation Guard

A new antd import or new .less file was added. Use UntitledUI + Tailwind for new work.

Affected files

at Function._resolveFilename (node:internal/modules/cjs/loader:1401:15)
at defaultResolveImpl (node:internal/modules/cjs/loader:1057:19)
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1062:22)
at Function._load (node:internal/modules/cjs/loader:1211:37)
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
at wrapModuleLoad (node:internal/modules/cjs/loader:235:24)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:171:5)
at node:internal/main/run_main_module:36:49 {


Fix locally (fast - only checks files changed in this branch):

make ui-checkstyle-changed

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Jest test Coverage

UI tests summary

Lines Statements Branches Functions
Coverage: 63%
63.71% (61572/96636) 44.64% (33424/74860) 46.76% (10125/21652)

@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Add this label to run secure Github workflows on PRs skip-pr-checks Bypass PR metadata validation check UI UI specific issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant