yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2025-11-01 11:09:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

COVERAGE: fix the coverage issue due to dataAssetRules changes (#22801)

Browse Source 
* fix the coverage issue due to dataAssetRules changes

* remove the commented code

(cherry picked from commit 7fa6518a551ff5a00dd7cf4e03baeeeca57cec19)
This commit is contained in:
Ashish Gupta 2025-08-07 01:00:54 +05:30 committed by OpenMetadata Release Bot
parent 555c599364
commit 39dcd0c0c6
1 changed files with 1 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
openmetadata-ui/src/main/resources/ui/src/components/DataAssetRules/DataAssetRules.component.tsx
View File

@ -321,14 +321,6 @@ export const useSemanticsRuleList = ({
});
};
const handleEditSemanticsRule = (semanticsRule: SemanticsRule) => {
setAddEditSemanticsRule(semanticsRule);
};
const handleDelete = (semanticsRule: SemanticsRule) => {
setDeleteSemanticsRule(semanticsRule);
};
const onConfirmDelete = async (semanticsRule: SemanticsRule) => {
const updatedSemanticsRules = semanticsRules.filter(
(rule) => rule.name !== semanticsRule.name
@ -363,29 +355,6 @@ export const useSemanticsRuleList = ({
/>
),
},
// Hidden for now - Actions column with edit and delete buttons
// {
// title: t('label.action'),
// dataIndex: 'actions',
// render: (_: unknown, record: SemanticsRule) => (
// <Space className="custom-icon-button">
// <Button
// className="text-secondary p-0 remove-button-background-hover"
// disabled={record.provider === ProviderType.System}
// icon={<Icon component={IconEdit} />}
// type="text"
// onClick={() => handleEditSemanticsRule(record)}
// />
// <Button
// className="text-secondary p-0 remove-button-background-hover"
// disabled={record.provider === ProviderType.System}
// icon={<Icon component={IconDelete} />}
// type="text"
// onClick={() => handleDelete(record)}
// />
// </Space>
// ),
// },
];
const dataAssetRuleList = useMemo(() => {
@ -440,12 +409,7 @@ export const useSemanticsRuleList = ({
);
return {
addSemanticsRuleButton: null, // Hidden for now
// addSemanticsRuleButton: (
// <Button type="primary" onClick={handleAddDataAssetRule}>
// {t('label.add-data-asset-rule')}
// </Button>
// ),
addSemanticsRuleButton: null,
semanticsRuleList: (
<>
{quickAddSemanticsRule || dataAssetRuleList}