Skip to content

Fix UI Suspense loader fallbacks#29746

Open
shah-harshit wants to merge 16 commits into
mainfrom
codex/fix-ui-suspense-loaders
Open

Fix UI Suspense loader fallbacks#29746
shah-harshit wants to merge 16 commits into
mainfrom
codex/fix-ui-suspense-loaders

address comments and fix checkstyle

7485779
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Jul 6, 2026 in 5m 57s

Annotations

Check warning on line 39 in src/hooks/useCustomPages.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83exzdHtxuMtvMzF7_&open=AZ83exzdHtxuMtvMzF7_&pullRequest=29746

Check warning on line 141 in src/utils/FeedUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

The signature '(element: FunctionComponentElement<any> | FunctionComponentElement<any>[], container: Container | null, callback?: (() => void) | undefined): void' of 'ReactDOM.render' is deprecated.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83eqS7HtxuMtvMzF1P&open=AZ83eqS7HtxuMtvMzF1P&pullRequest=29746

Check warning on line 244 in src/pages/ColumnBulkOperations/ColumnGrid/ColumnGrid.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Consider Array.prototype.some() instead of some() from lodash; check that the behavior is equivalent because the library also accepts objects, nullish values, and shorthand predicates.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ewEfHtxuMtvMzF6T&open=AZ83ewEfHtxuMtvMzF6T&pullRequest=29746

Check warning on line 160 in src/components/Settings/CustomProperty/CustomPropertyTable.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use Array.isArray() instead of isArray() from lodash.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ekF-HtxuMtvMzFwb&open=AZ83ekF-HtxuMtvMzFwb&pullRequest=29746

Check warning on line 128 in src/components/common/UserTeamSelectableList/UserTeamSelectableList.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.isNaN` over `isNaN`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83efrlHtxuMtvMzFs4&open=AZ83efrlHtxuMtvMzFs4&pullRequest=29746

Check warning on line 29 in src/utils/DataAssetsHeaderTypeUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use Array.isArray() instead of isArray() from lodash.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83etwqHtxuMtvMzF43&open=AZ83etwqHtxuMtvMzF43&pullRequest=29746

Check warning on line 429 in src/components/Entity/Task/TaskTab/TaskTabNew.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83elwoHtxuMtvMzFxu&open=AZ83elwoHtxuMtvMzFxu&pullRequest=29746

Check warning on line 247 in src/pages/DataInsightPage/DataInsightProvider.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83euswHtxuMtvMzF5h&open=AZ83euswHtxuMtvMzF5h&pullRequest=29746

Check warning on line 419 in src/components/ActivityFeed/ActivityThreadPanel/ActivityThreadPanelBody.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83eh1QHtxuMtvMzFur&open=AZ83eh1QHtxuMtvMzFur&pullRequest=29746

Check failure on line 72 in src/components/common/EntityDescription/EntityAttachmentProvider/EntityAttachmentProvider.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83efjnHtxuMtvMzFsw&open=AZ83efjnHtxuMtvMzFsw&pullRequest=29746

Check failure on line 65 in src/components/BlockEditor/Extensions/File/FileNode.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `getAttribute(…)`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ebsAHtxuMtvMzFp4&open=AZ83ebsAHtxuMtvMzFp4&pullRequest=29746

Check failure on line 127 in src/components/ExploreV1/ExploreV1.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83eoq4HtxuMtvMzF0G&open=AZ83eoq4HtxuMtvMzF0G&pullRequest=29746

Check warning on line 169 in src/components/Visualisations/Chart/CardinalityDistributionChart.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83epVyHtxuMtvMzF0d&open=AZ83epVyHtxuMtvMzF0d&pullRequest=29746

Check warning on line 35 in src/utils/EntityStatusUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unnecessary use of boolean literals in conditional expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ets6HtxuMtvMzF40&open=AZ83ets6HtxuMtvMzF40&pullRequest=29746

Check warning on line 544 in src/components/DataQuality/TestSuite/TestSuitePipelineTab/TestSuitePipelineTab.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83edKgHtxuMtvMzFq7&open=AZ83edKgHtxuMtvMzFq7&pullRequest=29746

Check warning on line 320 in src/utils/CSV/CSVUtilsClassBase.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

'value.fullyQualifiedName ?? ''' will use Object's default stringification format ('[object Object]') when stringified.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83eqt-HtxuMtvMzF1j&open=AZ83eqt-HtxuMtvMzF1j&pullRequest=29746

Check warning on line 289 in src/pages/AddNotificationPage/AddNotificationPage.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

arrow function is equivalent to `Boolean`. Use `Boolean` directly.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ewW3HtxuMtvMzF6u&open=AZ83ewW3HtxuMtvMzF6u&pullRequest=29746

Check warning on line 77 in src/components/Dashboard/DataModel/DataModels/ModelTab/ModelTab.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

useState call is not destructured into value + setter pair

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83eclSHtxuMtvMzFqa&open=AZ83eclSHtxuMtvMzFqa&pullRequest=29746

Check warning on line 68 in src/utils/DataInsightChartUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83esK7HtxuMtvMzF3I&open=AZ83esK7HtxuMtvMzF3I&pullRequest=29746

Check warning on line 52 in src/components/Entity/EntityVersionTimeLine/VersionButton.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Visible, non-interactive elements with click handlers must have at least one keyboard listener.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83el5OHtxuMtvMzFx0&open=AZ83el5OHtxuMtvMzFx0&pullRequest=29746

Check warning on line 304 in src/components/common/AsyncSelectList/TreeAsyncSelectList.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Provide multiple methods instead of using "isMultiSelect" to determine which action to take.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83efcHHtxuMtvMzFst&open=AZ83efcHHtxuMtvMzFst&pullRequest=29746

Check warning on line 406 in src/pages/ExplorePage/ExplorePageV1.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Consider removing 'undefined' type or '?' specifier, one of them is redundant.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ewq3HtxuMtvMzF69&open=AZ83ewq3HtxuMtvMzF69&pullRequest=29746

Check warning on line 138 in src/components/DataQuality/TestCases/useTestCaseList.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Consider Set instead of uniq() from lodash; for example, use `[...new Set(values)]`. Check that the behavior is equivalent because the library handles nullish values differently from the native API.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83edBeHtxuMtvMzFqz&open=AZ83edBeHtxuMtvMzFqz&pullRequest=29746

Check warning on line 144 in src/components/MyData/Widgets/KPIWidget/KPILegend/KPILegend.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Math.max()` to simplify ternary expressions.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83ehgkHtxuMtvMzFuf&open=AZ83ehgkHtxuMtvMzFuf&pullRequest=29746

Check warning on line 788 in src/utils/JSONLogicSearchClassBase.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ83euhyHtxuMtvMzF5c&open=AZ83euhyHtxuMtvMzF5c&pullRequest=29746