Skip to content

refactor(ui): split alerts into reusable hooks and components#29708

Merged
shah-harshit merged 7 commits into
mainfrom
codex/alerts-ui-hooks-refactor
Jul 3, 2026
Merged

refactor(ui): split alerts into reusable hooks and components#29708
shah-harshit merged 7 commits into
mainfrom
codex/alerts-ui-hooks-refactor

fix(ui): refetch alert details on fqn changes

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

Annotations

Check warning on line 719 in src/utils/ServiceConnectionUtils.ts

See this annotation in the file changed.

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

Review this redundant assignment: "section" already holds the assigned value along all execution paths.

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

Check warning on line 261 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=AZ8jSdF-RFoArOAaAUoj&open=AZ8jSdF-RFoArOAaAUoj&pullRequest=29708

Check warning on line 84 in src/utils/FeedUtilsPure.ts

See this annotation in the file changed.

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

Replace this union type with a type alias.

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

Check warning on line 43 in src/pages/TasksPage/shared/DiffViewNew.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=AZ8jSeHeRFoArOAaAUyQ&open=AZ8jSeHeRFoArOAaAUyQ&pullRequest=29708

Check failure on line 577 in src/pages/EntityVersionPage/EntityVersionPage.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 45 to the 15 allowed.

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

Check warning on line 80 in src/components/ActivityFeed/ActivityFeedEditor/ActivityFeedEditor.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=AZ8jScy4RFoArOAaAUmM&open=AZ8jScy4RFoArOAaAUmM&pullRequest=29708

Check failure on line 754 in src/components/ActivityFeed/ActivityFeedProvider/ActivityFeedProvider.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=AZ8jSc0lRFoArOAaAUmZ&open=AZ8jSc0lRFoArOAaAUmZ&pullRequest=29708

Check warning on line 93 in src/components/DataInsight/KPILatestResultsV1.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=AZ8jSdWrRFoArOAaAUqj&open=AZ8jSdWrRFoArOAaAUqj&pullRequest=29708

Check warning on line 116 in src/pages/TasksPage/RequestTagPage/RequestTagPage.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=AZ8jSeIYRFoArOAaAUyV&open=AZ8jSeIYRFoArOAaAUyV&pullRequest=29708

Check warning on line 103 in src/components/SettingsSso/SsoTestLogin/SsoTestLoginModal.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=AZ8jSdR7RFoArOAaAUp-&open=AZ8jSdR7RFoArOAaAUp-&pullRequest=29708

Check warning on line 85 in src/constants/Schedular.constants.ts

See this annotation in the file changed.

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

Use `new Array()` instead of `Array()`.

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

Check warning on line 144 in src/utils/KnowledgePagePureUtils.ts

See this annotation in the file changed.

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

Default parameters should be last.

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

Check warning on line 225 in src/components/ActivityFeed/TaskFeedCard/TaskFeedCard.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=AZ8jSc1LRFoArOAaAUmg&open=AZ8jSc1LRFoArOAaAUmg&pullRequest=29708

Check warning on line 225 in src/rest/miscAPI.ts

See this annotation in the file changed.

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

Arrow function has too many parameters (8). Maximum allowed is 7.

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

Check warning on line 125 in src/hooks/useGridEditController.ts

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=AZ8jSeckRFoArOAaAU0y&open=AZ8jSeckRFoArOAaAU0y&pullRequest=29708

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=AZ8jSeFGRFoArOAaAUyE&open=AZ8jSeFGRFoArOAaAUyE&pullRequest=29708

Check warning on line 862 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=AZ8jSeUoRFoArOAaAUzu&open=AZ8jSeUoRFoArOAaAUzu&pullRequest=29708

Check warning on line 102 in src/components/Domain/DomainVersion/DomainVersion.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=AZ8jScuURFoArOAaAUlo&open=AZ8jScuURFoArOAaAUlo&pullRequest=29708

Check warning on line 200 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=AZ8jSd6WRFoArOAaAUwT&open=AZ8jSd6WRFoArOAaAUwT&pullRequest=29708

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=AZ8jSdQVRFoArOAaAUpu&open=AZ8jSdQVRFoArOAaAUpu&pullRequest=29708

Check warning on line 42 in src/components/KnowledgeGraph3D/nodeCanvas.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=AZ8jScsgRFoArOAaAUlM&open=AZ8jScsgRFoArOAaAUlM&pullRequest=29708

Check warning on line 575 in src/components/OntologyExplorer/hooks/useGraphData.ts

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=AZ8jSdcTRFoArOAaAUrj&open=AZ8jSdcTRFoArOAaAUrj&pullRequest=29708

Check warning on line 209 in src/components/Metric/RelatedMetrics/RelatedMetrics.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=AZ8jSdfsRFoArOAaAUsJ&open=AZ8jSdfsRFoArOAaAUsJ&pullRequest=29708

Check warning on line 207 in src/utils/WorkflowSerializer.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=AZ8jSd-ORFoArOAaAUxH&open=AZ8jSd-ORFoArOAaAUxH&pullRequest=29708

Check warning on line 173 in src/components/common/EntityImport/CsvJobsTray/CsvJobsTray.component.tsx

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=AZ8jScZ1RFoArOAaAUjF&open=AZ8jScZ1RFoArOAaAUjF&pullRequest=29708