Skip to content

Backport #29179 entity detail lazy-loading utilities to 1.13 - #31050

Open
shah-harshit wants to merge 1 commit into
ui/direct-1.13-seq33-29178-activity-feed-editor-formattingfrom
ui/direct-1.13-seq34-29179-entity-detail-lazy-utils
Open

Backport #29179 entity detail lazy-loading utilities to 1.13#31050
shah-harshit wants to merge 1 commit into
ui/direct-1.13-seq33-29178-activity-feed-editor-formattingfrom
ui/direct-1.13-seq34-29179-entity-detail-lazy-utils

Conversation

@shah-harshit

Copy link
Copy Markdown
Contributor

Summary

Tracking

Testing

  • git diff --check HEAD~1..HEAD

* perf(ui): lazy load activity feed and split editor formatting (#29178)

* perf(ui): lazy load activity feed editor surfaces

* fix(ui): remove activity feed cache split dependencies

* fix(ui): apply activity feed checkstyle

* fix(ui): consolidate block editor content formatting

* fix(ui): clarify block editor server formatting

* perf(ui): lazy load entity detail pages and utilities (#29179)

* perf(ui): lazy load entity detail components

* fix(ui): resolve entity detail lazy split imports

* fix(ui): apply entity lazy checkstyle

(cherry picked from commit ae8744a)
@shah-harshit
shah-harshit requested a review from a team as a code owner August 5, 2026 13:34
@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
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

❌ PR checklist incomplete

This PR cannot be merged until the following are addressed on its linked issue:

  • No GitHub issue is linked. Link an issue in the Development section of the PR (or add Fixes #12345 to the description). For a same-org cross-repo issue, add Fixes open-metadata/<repo>#123 to the description.

The fields live on the linked issue in the Shipping project (open the issue → right sidebar → Projects). After you set them, re-run this check (or push a commit) — issue/project changes do not re-trigger it automatically.

Maintainers can bypass this check by adding the skip-pr-checks label.

@gitar-bot

gitar-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Backports entity detail lazy-loading utilities to the 1.13 branch as part of the stacked sequence. 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

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