Skip to content

(feat)ui: migrate form builder in connection form #27812

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

fix failing sonar

a37d2d6
Select commit
Loading
Failed to load commit list.
Open

(feat)ui: migrate form builder in connection form #27812

fix failing sonar
a37d2d6
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 4, 2026 in 5m 25s

Quality Gate passed

Issues
1078 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 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=AZ3ZmObKxqhRDr6QzuPd&open=AZ3ZmObKxqhRDr6QzuPd&pullRequest=27812

Check warning on line 130 in src/components/common/UserTeamSelectableList/UserTeamSelectableList.component.tsx

See this annotation in the file changed.

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

Prefer `Number.isNaN` over `isNaN`. 

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

Check warning on line 77 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=AZ3ZmNujxqhRDr6QzuOA&open=AZ3ZmNujxqhRDr6QzuOA&pullRequest=27812

Check failure on line 750 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=AZ3ZmPGjxqhRDr6QzuQr&open=AZ3ZmPGjxqhRDr6QzuQr&pullRequest=27812

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=AZ3ZmPUaxqhRDr6QzuRG&open=AZ3ZmPUaxqhRDr6QzuRG&pullRequest=27812

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

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=AZ3ZmTDnxqhRDr6QzuZm&open=AZ3ZmTDnxqhRDr6QzuZm&pullRequest=27812

Check warning on line 1491 in src/utils/EntityLineageUtils.tsx

See this annotation in the file changed.

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

Spread an object literal in object literal is unnecessary. 

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

Check warning on line 114 in src/components/WorkflowDefinitions/WorkflowBuilder/NodeConfigSidebar.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=AZ3ZmPbxxqhRDr6QzuRd&open=AZ3ZmPbxxqhRDr6QzuRd&pullRequest=27812

Check warning on line 148 in src/components/common/UserSelectableList/UserSelectableList.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=AZ3ZmOKYxqhRDr6QzuPF&open=AZ3ZmOKYxqhRDr6QzuPF&pullRequest=27812

Check failure on line 333 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=AZ3ZmTY1xqhRDr6Qzuay&open=AZ3ZmTY1xqhRDr6Qzuay&pullRequest=27812

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=AZ3ZmR1VxqhRDr6QzuWV&open=AZ3ZmR1VxqhRDr6QzuWV&pullRequest=27812

Check warning on line 172 in src/context/PermissionProvider/PermissionProvider.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=AZ3ZmT5CxqhRDr6Qzucy&open=AZ3ZmT5CxqhRDr6Qzucy&pullRequest=27812

Check warning on line 989 in src/pages/ColumnBulkOperations/ColumnGrid/ColumnGrid.component.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=AZ3ZmSK3xqhRDr6QzuXC&open=AZ3ZmSK3xqhRDr6QzuXC&pullRequest=27812

Check warning on line 122 in src/pages/DataQuality/DataQualityProvider.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=AZ3ZmSRUxqhRDr6QzuXX&open=AZ3ZmSRUxqhRDr6QzuXX&pullRequest=27812

Check failure on line 354 in src/utils/TaskFormSchemaRegistry.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=AZ3ZmTFyxqhRDr6QzuZ4&open=AZ3ZmTFyxqhRDr6QzuZ4&pullRequest=27812

Check warning on line 24 in src/utils/EdgeStyleUtils.ts

See this annotation in the file changed.

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

Function 'calculateEdgeStyle' has too many parameters (9). Maximum allowed is 7. 

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

Check failure on line 118 in src/hooks/useWorkflowEdgeManagement.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=AZ3ZmTu-xqhRDr6QzucC&open=AZ3ZmTu-xqhRDr6QzucC&pullRequest=27812

Check warning on line 27 in src/utils/SwTokenStorageUtils.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=AZ3ZmSylxqhRDr6QzuYW&open=AZ3ZmSylxqhRDr6QzuYW&pullRequest=27812

Check failure on line 758 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=AZ3ZmOjUxqhRDr6QzuPw&open=AZ3ZmOjUxqhRDr6QzuPw&pullRequest=27812

Check failure on line 469 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 42 to the 15 allowed. 

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

Check warning on line 330 in src/utils/WorkflowNodeConfigUtils.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=AZ3ZmSzSxqhRDr6QzuYb&open=AZ3ZmSzSxqhRDr6QzuYb&pullRequest=27812

Check warning on line 47 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=AZ3ZmNGyxqhRDr6QzuMe&open=AZ3ZmNGyxqhRDr6QzuMe&pullRequest=27812

Check warning on line 13 in src/components/Modals/DeleteModal/DeleteModal.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=AZ3ZmOaaxqhRDr6QzuPc&open=AZ3ZmOaaxqhRDr6QzuPc&pullRequest=27812

Check warning on line 49 in src/components/Auth/AppAuthenticators/MsalAuthenticator.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=AZ3ZmQ3SxqhRDr6QzuUA&open=AZ3ZmQ3SxqhRDr6QzuUA&pullRequest=27812

Check failure on line 66 in src/components/BlockEditor/Extensions/File/FileNode.ts

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=AZ3ZmMtixqhRDr6QzuL4&open=AZ3ZmMtixqhRDr6QzuL4&pullRequest=27812
View more details on SonarQubeCloud