Skip to content

refactor: replace RouterUtils with ObservabilityRouterClassBase for navigation paths #27956

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
ShaileshParmar11 wants to merge 6 commits into
base: main
Choose a base branch
from

merged main into branch

21f0089
Select commit
Loading
Failed to load commit list.
Open

refactor: replace RouterUtils with ObservabilityRouterClassBase for navigation paths #27956

merged main into branch
21f0089
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 9, 2026 in 6m 14s

Quality Gate passed

Issues
1353 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 165 in src/components/DataQuality/AddDataQualityTest/components/EditTestCaseModalV1.tsx

See this annotation in the file changed.

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

Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element. 

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

Check warning on line 54 in src/components/Settings/Team/TeamsSelectable/TeamsSelectable.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=AZ4B2x_E3F1zEi-E8RKP&open=AZ4B2x_E3F1zEi-E8RKP&pullRequest=27956

Check warning on line 649 in src/components/Settings/CustomProperty/AddCustomProperty/AddCustomProperty.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=AZ4H8GvzpJDH8099FslO&open=AZ4H8GvzpJDH8099FslO&pullRequest=27956

Check failure on line 941 in src/utils/QueryBuilderElasticsearchFormatUtils.js

See this annotation in the file changed.

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

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

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

Check failure on line 603 in src/components/OntologyExplorer/hooks/useOntologyExplorer.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 16 to the 15 allowed. 

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

Check warning on line 253 in src/components/common/FileUpload/MUIFileUpload.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=AZ4B2uAi3F1zEi-E8RGN&open=AZ4B2uAi3F1zEi-E8RGN&pullRequest=27956

Check warning on line 627 in src/components/Settings/CustomProperty/AddCustomProperty/AddCustomProperty.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=AZ4H8GvzpJDH8099FslN&open=AZ4H8GvzpJDH8099FslN&pullRequest=27956

Check warning on line 112 in src/pages/Glossary/GlossaryLeftPanel/GlossaryLeftPanel.component.tsx

See this annotation in the file changed.

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

'LeftPanelCard' is deprecated. 

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

Check warning on line 204 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=AZ4B21d23F1zEi-E8RN-&open=AZ4B21d23F1zEi-E8RN-&pullRequest=27956

Check warning on line 79 in src/components/common/AsyncSelect/AsyncSelect.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=AZ4B2u0c3F1zEi-E8RG8&open=AZ4B2u0c3F1zEi-E8RG8&pullRequest=27956

Check failure on line 45 in src/components/common/MUIAsyncTreeSelect/hooks/useTreeKeyboardNavigation.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 52 to the 15 allowed. 

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

Check warning on line 106 in src/components/WorkflowDefinitions/WorkflowBuilder/NodeConfigSidebar.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=AZ4B2xGL3F1zEi-E8RJM&open=AZ4B2xGL3F1zEi-E8RJM&pullRequest=27956

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

ReactDOM.render is deprecated since React 18.0.0, use createRoot instead 

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

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=AZ4B2v413F1zEi-E8RH6&open=AZ4B2v413F1zEi-E8RH6&pullRequest=27956

Check warning on line 243 in src/pages/APICollectionPage/APICollectionPage.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=AZ4H8IStpJDH8099FsmM&open=AZ4H8IStpJDH8099FsmM&pullRequest=27956

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=AZ4B23hU3F1zEi-E8RQf&open=AZ4B23hU3F1zEi-E8RQf&pullRequest=27956

Check warning on line 407 in src/utils/CommonUtils.tsx

See this annotation in the file changed.

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

Prefer `String#codePointAt()` over `String#charCodeAt()`. 

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

Check warning on line 29 in src/components/ActivityFeed/ActivityFeedCardNew/TaskCommentCard.component.tsx

See this annotation in the file changed.

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

'task' PropType is defined but prop is never used 

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

Check warning on line 131 in src/utils/WorkflowConfigUtils.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=AZ4B23lh3F1zEi-E8RQk&open=AZ4B23lh3F1zEi-E8RQk&pullRequest=27956

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=AZ4B25H13F1zEi-E8RTL&open=AZ4B25H13F1zEi-E8RTL&pullRequest=27956

Check warning on line 18 in src/components/KnowledgeCenter/KnowledgePagesHierarchy/KnowledgePagesHierarchy.tsx

See this annotation in the file changed.

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

'../../../constants/constants' imported multiple times. 

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

Check warning on line 38 in src/components/Modals/ProfileEditModal/ProfileEditModal.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=AZ4B2vO03F1zEi-E8RHR&open=AZ4B2vO03F1zEi-E8RHR&pullRequest=27956

Check warning on line 161 in src/components/WorkflowDefinitions/WorkflowBuilder/forms/UserApprovalForm.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=AZ4B2w5K3F1zEi-E8RI6&open=AZ4B2w5K3F1zEi-E8RI6&pullRequest=27956

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

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=AZ4B2y8W3F1zEi-E8RLH&open=AZ4B2y8W3F1zEi-E8RLH&pullRequest=27956

Check warning on line 608 in src/components/DataAssetSummaryPanelV1/DataAssetSummaryPanelV1.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=AZ4H8H-NpJDH8099Fsl7&open=AZ4H8H-NpJDH8099Fsl7&pullRequest=27956
View more details on SonarQubeCloud