Skip to content

refactor(DimensionalityTab): migrate table to core-ui Table component #27969

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
shah-harshit wants to merge 12 commits into
base: main
Choose a base branch
from

Merge branch 'main' into feat/migrate-dimensionality-tab-core-ui-table

fc56967
Select commit
Loading
Failed to load commit list.
Open

refactor(DimensionalityTab): migrate table to core-ui Table component #27969

Merge branch 'main' into feat/migrate-dimensionality-tab-core-ui-table
fc56967
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 8, 2026 in 6m 9s

Quality Gate passed

Issues
1290 New issues
0 Accepted issues

Measures
17 Security Hotspots
0.0% Coverage on New Code
3.8% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 246 in src/utils/TagClassBase.ts

See this annotation in the file changed.

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

This assertion is unnecessary since the receiver accepts the original type of the expression. 

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

Check warning on line 183 in src/components/Settings/Users/UsersProfile/UserPermissions/UserPermissions.component.tsx

See this annotation in the file changed.

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

Do not use Array index in keys 

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

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

See this annotation in the file changed.

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

Unexpected negated condition. 

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

Check warning on line 886 in src/pages/GlossaryTermRelationSettings/GlossaryTermRelationSettings.tsx

See this annotation in the file changed.

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

Prefer `Number.parseInt` over `parseInt`. 

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

Check warning on line 292 in src/utils/TasksUtils.ts

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 201 in src/components/DomainListing/components/DomainTreeView.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since the receiver accepts the original type of the expression. 

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

Check warning on line 77 in src/components/Glossary/GlossaryTerms/tabs/GlossaryTermReferences.tsx

See this annotation in the file changed.

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

Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read. 

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

Check warning on line 118 in src/utils/ServiceConnectionDetailsUtils.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 591 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=AZ4DBNcBYfl7KKnbx8Vo&open=AZ4DBNcBYfl7KKnbx8Vo&pullRequest=27969

Check warning on line 85 in src/components/common/Form/JSONSchema/JSONSchemaTemplate/WorkflowArrayFieldTemplate.tsx

See this annotation in the file changed.

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

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

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

Check warning on line 536 in src/utils/CanvasUtils.ts

See this annotation in the file changed.

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

Prefer `Number.parseFloat` over `parseFloat`. 

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

Check warning on line 207 in src/rest/suggestionsAPI.ts

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=AZ4DBTFuYfl7KKnbx8e6&open=AZ4DBTFuYfl7KKnbx8e6&pullRequest=27969

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

See this annotation in the file changed.

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

'render' is deprecated. 

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

Check failure on line 104 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 18 to the 15 allowed. 

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

Check warning on line 122 in src/components/Settings/Users/UsersProfile/UserPermissions/UserPermissions.component.tsx

See this annotation in the file changed.

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

Do not use Array index in keys 

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

Check warning on line 144 in src/pages/TasksPage/UpdateDescriptionPage/UpdateDescriptionPage.tsx

See this annotation in the file changed.

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

Promise-returning function provided to variable where a void return was expected. 

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

Check warning on line 54 in src/components/WorkflowDefinitions/WorkflowBuilder/WorkflowExecutionHistory.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 137 in src/pages/TasksPage/shared/TaskPayloadSchemaFields.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 124 in src/pages/EditConnectionFormPage/EditConnectionFormPage.component.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 473 in src/components/Customization/GenericProvider/GenericProvider.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since the receiver accepts the original type of the expression. 

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

Check warning on line 251 in src/pages/EditConnectionFormPage/EditConnectionFormPage.component.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 60 in src/components/MyData/CustomizableComponents/CustomiseGlossaryTermDetailPage/CustomiseGlossaryTermDetailPage.tsx

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 375 in src/components/Settings/Applications/AppLogsViewer/AppLogsViewer.component.tsx

See this annotation in the file changed.

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

Unexpected negated condition. 

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

Check warning on line 476 in src/pages/TasksPage/shared/TaskPayloadSchemaFields.tsx

See this annotation in the file changed.

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

'getFieldValue(fieldName, '') ?? ''' will use Object's default stringification format ('[object Object]') when stringified. 

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

Check warning on line 357 in src/components/common/CoverImageUpload/MUICoverImageUpload.tsx

See this annotation in the file changed.

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

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4DBIdfYfl7KKnbx8PQ&open=AZ4DBIdfYfl7KKnbx8PQ&pullRequest=27969
View more details on SonarQubeCloud