Skip to content

Fixed the component re-rendering issue which causes the Entity test failure #27355

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

Merged
harshach merged 4 commits into from
Apr 14, 2026

fixed loading issue

0bab0e1
Select commit
Loading
Failed to load commit list.
Merged

Fixed the component re-rendering issue which causes the Entity test failure #27355

fixed loading issue
0bab0e1
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Apr 14, 2026 in 7m 44s

Quality Gate passed

Issues
1092 New issues
0 Accepted issues

Measures
14 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 100 in src/pages/DataInsightPage/DataInsightHeader/DataInsightHeader.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=AZ2NCLrUgoR4mEhKMXYY&open=AZ2NCLrUgoR4mEhKMXYY&pullRequest=27355

Check warning on line 120 in src/components/BlockEditor/Extensions/BlockAndDragDrop/BlockAndDragHandle.ts

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=AZ2NCKKugoR4mEhKMXPM&open=AZ2NCKKugoR4mEhKMXPM&pullRequest=27355

Check warning on line 70 in src/pages/RolesPage/RolesDetailPage/RolesDetailPage.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=AZ2NCLrvgoR4mEhKMXYZ&open=AZ2NCLrvgoR4mEhKMXYZ&pullRequest=27355

Check warning on line 286 in src/pages/LearningResourcesPage/LearningResourceForm.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=AZ2NCLyMgoR4mEhKMXYw&open=AZ2NCLyMgoR4mEhKMXYw&pullRequest=27355

Check warning on line 209 in src/utils/StringsUtils.ts

See this annotation in the file changed.

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

`String.raw` should be used to avoid escaping `\`. 

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

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=AZ2NCLnOgoR4mEhKMXYI&open=AZ2NCLnOgoR4mEhKMXYI&pullRequest=27355

Check warning on line 284 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=AZ2NCK_1goR4mEhKMXUD&open=AZ2NCK_1goR4mEhKMXUD&pullRequest=27355

Check failure on line 574 in src/components/Auth/AuthProviders/AuthProvider.tsx

See this annotation in the file changed.

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

Refactor this code to not nest functions more than 4 levels deep. 

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

Check warning on line 16 in src/utils/EnvironmentUtils.ts

See this annotation in the file changed.

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

Compare with `undefined` directly instead of using `typeof`. 

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

Check failure on line 224 in src/rest/columnAPI.ts

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=AZ2NCMZagoR4mEhKMXfc&open=AZ2NCMZagoR4mEhKMXfc&pullRequest=27355

Check warning on line 25 in src/pages/AppInstall/AppInstall.component.tsx

See this annotation in the file changed.

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

Prefer using the default import over named import. 

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

Check warning on line 122 in src/components/Settings/Services/AddIngestion/AddIngestion.component.tsx

See this annotation in the file changed.

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

The empty object is useless. 

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

Check warning on line 44 in src/components/UploadFile/UploadFile.tsx

See this annotation in the file changed.

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

Prefer `Blob#text()` over `FileReader#readAsText(blob)`. 

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

Check warning on line 178 in src/components/MyData/CustomizableComponents/CustomiseHomeModal/CustomiseHomeModal.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=AZ2NCK1OgoR4mEhKMXTO&open=AZ2NCK1OgoR4mEhKMXTO&pullRequest=27355

Check warning on line 146 in src/constants/Services.constant.ts

See this annotation in the file changed.

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

Use `export…from` to re-export `ORACLE`. 

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

Check warning on line 143 in src/components/OntologyExplorer/utils/graphStyles.ts

See this annotation in the file changed.

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

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

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

Check warning on line 51 in src/components/SearchSettings/FieldValueBoostList/FieldValueBoostList.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=AZ2NCLlYgoR4mEhKMXX_&open=AZ2NCLlYgoR4mEhKMXX_&pullRequest=27355

Check warning on line 247 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=AZ2NCL6ogoR4mEhKMXZY&open=AZ2NCL6ogoR4mEhKMXZY&pullRequest=27355

Check warning on line 48 in src/utils/SwMessenger.ts

See this annotation in the file changed.

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

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

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

Check warning on line 865 in src/components/common/Table/TableV2.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=AZ2NCKU0goR4mEhKMXQX&open=AZ2NCKU0goR4mEhKMXQX&pullRequest=27355

Check warning on line 79 in src/hooks/useClipBoard.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=AZ2NCMcBgoR4mEhKMXft&open=AZ2NCMcBgoR4mEhKMXft&pullRequest=27355

Check warning on line 213 in src/components/Glossary/GlossaryHeader/GlossaryHeader.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=AZ2NCLI6goR4mEhKMXU1&open=AZ2NCLI6goR4mEhKMXU1&pullRequest=27355

Check failure on line 436 in src/pages/ColumnBulkOperations/ColumnGrid/hooks/useColumnGridListingData.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 21 to the 15 allowed. 

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

Check failure on line 452 in src/components/LineageTable/LineageTable.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 19 to the 15 allowed. 

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

Check warning on line 204 in src/constants/Services.constant.ts

See this annotation in the file changed.

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

Use `export…from` to re-export `CUSTOM_DRIVE_DEFAULT`. 

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