Skip to content

address comments

324263a
Select commit
Loading
Failed to load commit list.
Merged

Storage Service Connect , path specification #27408

address comments
324263a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Apr 21, 2026 in 5m 15s

Annotations

Check warning on line 178 in src/components/common/MUIUserTeamSelect/MUIUserTeamSelect.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=AZ2X0esHHyVWmhAefr5u&open=AZ2X0esHHyVWmhAefr5u&pullRequest=27408

Check warning on line 87 in src/pages/PoliciesPage/PoliciesDetailPage/PoliciesDetailPage.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=AZ2X0gZ9HyVWmhAefsAZ&open=AZ2X0gZ9HyVWmhAefsAZ&pullRequest=27408

Check warning on line 153 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=AZ2X0gQ2HyVWmhAefsAB&open=AZ2X0gQ2HyVWmhAefsAB&pullRequest=27408

Check warning on line 257 in src/components/ActivityFeed/ActivityThreadPanel/ActivityThreadPanelBody.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=AZ2X0fJqHyVWmhAefr7W&open=AZ2X0fJqHyVWmhAefr7W&pullRequest=27408

Check warning on line 253 in src/rest/rdfAPI.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=AZ2X0hY8HyVWmhAefsHZ&open=AZ2X0hY8HyVWmhAefsHZ&pullRequest=27408

Check warning on line 93 in src/components/MyData/CustomizableComponents/CustomiseLandingPageHeader/CustomiseSearchBar.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=AZ2X0fAwHyVWmhAefr61&open=AZ2X0fAwHyVWmhAefr61&pullRequest=27408

Check warning on line 154 in src/rest/rdfAPI.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=AZ2X0hY8HyVWmhAefsHQ&open=AZ2X0hY8HyVWmhAefsHQ&pullRequest=27408

Check warning on line 364 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=AZ2X0eQfHyVWmhAefr3m&open=AZ2X0eQfHyVWmhAefr3m&pullRequest=27408

Check warning on line 149 in src/components/common/FileUpload/MUIFileUpload.tsx

See this annotation in the file changed.

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

'If' statement should not be the only statement in 'else' block

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

Check warning on line 155 in src/rest/rdfAPI.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=AZ2X0hY8HyVWmhAefsHR&open=AZ2X0hY8HyVWmhAefsHR&pullRequest=27408

Check warning on line 104 in src/components/DataContract/ContractSecurity/ContractSecurityCard.component.tsx

See this annotation in the file changed.

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

Missing "key" prop for element in iterator

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ2X0f-lHyVWmhAefr-h&open=AZ2X0f-lHyVWmhAefr-h&pullRequest=27408

Check warning on line 80 in src/pages/CreateUserPage/CreateUserPage.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=AZ2X0gePHyVWmhAefsAw&open=AZ2X0gePHyVWmhAefsAw&pullRequest=27408

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

See this annotation in the file changed.

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

Use the "RegExp.exec()" method instead.

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

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=AZ2X0gIWHyVWmhAefr_i&open=AZ2X0gIWHyVWmhAefr_i&pullRequest=27408

Check warning on line 87 in src/components/common/PopOverCard/EntityPopOverCard.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=AZ2X0eszHyVWmhAefr5y&open=AZ2X0eszHyVWmhAefr5y&pullRequest=27408

Check warning on line 40 in src/components/WorkflowDefinitions/Workflows/Nodes/GatewayNode/GatewayNode.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=AZ2ux8EAuBRwFKJ4akHN&open=AZ2ux8EAuBRwFKJ4akHN&pullRequest=27408

Check warning on line 285 in src/components/Entity/EntityLineage/CustomNode.utils.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=AZ2X0foaHyVWmhAefr86&open=AZ2X0foaHyVWmhAefr86&pullRequest=27408

Check failure on line 153 in src/components/SettingsSso/SSODocPanel/SSODocPanel.tsx

See this annotation in the file changed.

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

Prefer `.dataset` over `getAttribute(…)`.

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

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

See this annotation in the file changed.

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

Use the "RegExp.exec()" method instead.

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

Check warning on line 385 in src/components/Explore/ExploreTree/ExploreTree.tsx

See this annotation in the file changed.

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

Move this component definition out of the parent component and pass data as props.

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

Check warning on line 87 in src/components/common/atoms/TagChip/TagChip.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=AZ2X0eLnHyVWmhAefr3N&open=AZ2X0eLnHyVWmhAefr3N&pullRequest=27408

Check warning on line 15 in src/components/common/MUIAsyncTreeSelect/MUIAsyncTreeSelect.tsx

See this annotation in the file changed.

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

'useTreeViewApiRef' is deprecated.

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

Check warning on line 215 in src/components/common/NavigationBlocker/NavigationBlocker.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=AZ2X0egXHyVWmhAefr5C&open=AZ2X0egXHyVWmhAefr5C&pullRequest=27408

Check warning on line 112 in src/components/Entity/EntityLineage/CustomNode.utils.tsx

See this annotation in the file changed.

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

onMouseOut must be accompanied by onBlur for accessibility.

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

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

See this annotation in the file changed.

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

Prefer `.some(…)` over `.find(…)`.

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