Skip to content

Migrate IncidentManager table to core-ui Table component #27972

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
shah-harshit merged 9 commits into from
May 9, 2026
+192 −161

Fix incident manager tests

11e13ef
Select commit
Loading
Failed to load commit list.
Merged

Migrate IncidentManager table to core-ui Table component #27972

Fix incident manager tests
11e13ef
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 8, 2026 in 5m 57s

Quality Gate passed

Issues
1293 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 294 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=AZ4DdvP8N4tB5aUAqWQT&open=AZ4DdvP8N4tB5aUAqWQT&pullRequest=27972

Check warning on line 93 in src/constants/regex.constants.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=AZ4DdxXuN4tB5aUAqWUF&open=AZ4DdxXuN4tB5aUAqWUF&pullRequest=27972

Check failure on line 755 in src/components/DataQuality/AddDataQualityTest/components/TestCaseFormV1.tsx

See this annotation in the file changed.

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

Prefer `return value` over `return Promise.resolve(value)`. 

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

Check warning on line 255 in src/components/LineageTable/LineageTable.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=AZ4HPc24xQhg7-pf4M6-&open=AZ4HPc24xQhg7-pf4M6-&pullRequest=27972

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=AZ4DdrdPN4tB5aUAqWLG&open=AZ4DdrdPN4tB5aUAqWLG&pullRequest=27972

Check warning on line 385 in src/utils/WorkflowSerializer.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=AZ4DdwdeN4tB5aUAqWSk&open=AZ4DdwdeN4tB5aUAqWSk&pullRequest=27972

Check failure on line 306 in src/utils/SchedularUtils.tsx

See this annotation in the file changed.

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

Prefer `throw error` over `return Promise.reject(error)`. 

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

Check warning on line 748 in src/components/OntologyExplorer/hooks/useOntologyGraph.ts

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=AZ4DdtcxN4tB5aUAqWNr&open=AZ4DdtcxN4tB5aUAqWNr&pullRequest=27972

Check warning on line 13 in src/components/common/DeleteModal/DeleteModalMUI.tsx

See this annotation in the file changed.

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

Complete the task associated to this "TODO" comment. 

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

Check warning on line 425 in src/components/OntologyExplorer/hooks/useGraphData.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=AZ4DdtZmN4tB5aUAqWNi&open=AZ4DdtZmN4tB5aUAqWNi&pullRequest=27972

Check failure on line 1344 in src/components/OntologyExplorer/hooks/useOntologyExplorer.ts

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=AZ4DdthkN4tB5aUAqWN8&open=AZ4DdthkN4tB5aUAqWN8&pullRequest=27972

Check failure on line 62 in src/components/common/FileUpload/MUIFileUpload.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 25 to the 15 allowed. 

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

Check warning on line 326 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=AZ4HPchgxQhg7-pf4M6c&open=AZ4HPchgxQhg7-pf4M6c&pullRequest=27972

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

See this annotation in the file changed.

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

Use `.indexOf()` instead of `.findIndex()` when looking for the index of an item. 

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

Check warning on line 88 in src/pages/CustomizableDomainPage/CustomizableDomainPage.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=AZ4HPdOMxQhg7-pf4M7z&open=AZ4HPdOMxQhg7-pf4M7z&pullRequest=27972

Check failure on line 275 in src/components/MyData/Widgets/KPIWidget/KPIWidget.component.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=AZ4DdpotN4tB5aUAqWIu&open=AZ4DdpotN4tB5aUAqWIu&pullRequest=27972

Check warning on line 291 in src/components/SettingsSso/SSOGroupedFieldTemplate/SSOGroupedFieldTemplate.tsx

See this annotation in the file changed.

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

This branch's code block is the same as the block for the branch on line 279. 

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

Check warning on line 95 in src/pages/CustomizableDomainPage/CustomizableDomainPage.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=AZ4DdvCXN4tB5aUAqWP_&open=AZ4DdvCXN4tB5aUAqWP_&pullRequest=27972

Check failure on line 1374 in src/components/OntologyExplorer/hooks/useOntologyGraph.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 35 to the 15 allowed. 

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

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

See this annotation in the file changed.

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

Simplify this regular expression to reduce its complexity from 24 to the 20 allowed. 

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

Check warning on line 76 in src/utils/QueryBuilderElasticsearchFormatUtils.js

See this annotation in the file changed.

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

Complete the task associated to this "TODO" comment. 

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

Check warning on line 351 in src/utils/GlossaryUtils.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=AZ4DdxQLN4tB5aUAqWT5&open=AZ4DdxQLN4tB5aUAqWT5&pullRequest=27972

Check warning on line 103 in src/components/SettingsSso/SSOGroupedFieldTemplate/SSOGroupedFieldTemplate.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=AZ4DdsF8N4tB5aUAqWL0&open=AZ4DdsF8N4tB5aUAqWL0&pullRequest=27972

Check warning on line 158 in src/components/Database/TableQueries/QueryUsedByOtherTable/QueryUsedByOtherTable.component.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=AZ4DdmuDN4tB5aUAqWE3&open=AZ4DdmuDN4tB5aUAqWE3&pullRequest=27972

Check warning on line 30 in src/components/Auth/AppAuthenticators/GenericAuthenticator.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=AZ4Ddsq6N4tB5aUAqWMe&open=AZ4Ddsq6N4tB5aUAqWMe&pullRequest=27972
View more details on SonarQubeCloud