Impl [Artifacts] Delete artifact without a tag

src/api/artifacts-api.js

@@ -56,11 +56,10 @@ const fetchArtifacts = (project, filters, config = {}, withLatestTag) => {
 const artifactsApi = {
   addTag: (project, tag, data) => mainHttpClient.put(`/projects/${project}/tags/${tag}`, data),
   buildFunction: data => mainHttpClient.post('/build/function', data),
-  deleteArtifact: (project, key, tag, tree, deletion_strategy, secrets) => {
+  deleteArtifact: (project, key, uid, deletion_strategy, secrets) => {
     const config = {
       params: {
-        tag,
-        tree
+        'object-uid': uid
       }
     }
 

src/components/Datasets/datasets.util.js

@@ -306,11 +306,7 @@ export const generateActionsMenu = (
       {
         label: 'Delete',
         icon: <Delete />,
-        disabled: !datasetMin?.tag,
         hidden: [ACTION_MENU_PARENT_ROW, ACTION_MENU_PARENT_ROW_EXPANDED].includes(menuPosition),
-        tooltip: !datasetMin?.tag
-          ? 'A tag is required to delete a dataset. Open the dataset, click on the edit icon, and assign a tag before proceeding with the deletion'
-          : '',
         className: 'danger',
         onClick: () =>
           datasetDataCouldBeDeleted ?
@@ -329,8 +325,7 @@ export const generateActionsMenu = (
                   dispatch,
                   projectName,
                   datasetMin.db_key,
-                  datasetMin.tag,
-                  datasetMin.tree,
+                  datasetMin.uid,
                   handleRefresh,
                   datasetsFilters,
                   DATASET_TYPE
@@ -352,8 +347,7 @@ export const generateActionsMenu = (
                 dispatch,
                 projectName,
                 datasetMin.db_key,
-                datasetMin.tag,
-                datasetMin.tree,
+                datasetMin.uid,
                 handleRefresh,
                 datasetsFilters,
                 DATASET_TYPE,

src/components/Files/files.util.js

@@ -279,10 +279,6 @@ export const generateActionsMenu = (
         label: 'Delete',
         icon: <Delete />,
         hidden: [ACTION_MENU_PARENT_ROW, ACTION_MENU_PARENT_ROW_EXPANDED].includes(menuPosition),
-        disabled: !fileMin?.tag,
-        tooltip: !fileMin?.tag
-          ? 'A tag is required to delete an artifact. Open the artifact, click on the edit icon, and assign a tag before proceeding with the deletion'
-          : '',
         className: 'danger',
         onClick: () =>
           openPopUp(DeleteArtifactPopUp, {
@@ -307,8 +303,7 @@ export const generateActionsMenu = (
                 dispatch,
                 projectName,
                 fileMin.db_key,
-                fileMin.tag,
-                fileMin.tree,
+                fileMin.uid,
                 handleRefresh,
                 filters,
                 ARTIFACT_TYPE,

src/components/ModelsPage/Models/models.util.js

@@ -363,11 +363,7 @@ export const generateActionsMenu = (
         label: 'Delete',
         icon: <Delete />,
         className: 'danger',
-        disabled: !modelMin?.tag,
         hidden: [ACTION_MENU_PARENT_ROW, ACTION_MENU_PARENT_ROW_EXPANDED].includes(menuPosition),
-        tooltip: !modelMin?.tag
-          ? 'A tag is required to delete a model. Open the model, click on the edit icon, and assign a tag before proceeding with the deletion'
-          : '',
         onClick: () =>
           openDeleteConfirmPopUp(
             'Delete model?',
@@ -377,8 +373,7 @@ export const generateActionsMenu = (
                 dispatch,
                 projectName,
                 modelMin.db_key,
-                modelMin.tag,
-                modelMin.tree,
+                modelMin.uid,
                 handleRefresh,
                 modelsFilters,
                 MODEL_TYPE
@@ -400,8 +395,7 @@ export const generateActionsMenu = (
                 dispatch,
                 projectName,
                 modelMin.db_key,
-                modelMin.tag,
-                modelMin.tree,
+                modelMin.uid,
                 handleRefresh,
                 modelsFilters,
                 MODEL_TYPE,

src/elements/DeleteArtifactPopUp/DeleteArtifactPopUp.js

@@ -74,8 +74,7 @@ const DeleteArtifactPopUp = ({ artifact, artifactType, category, filters, handle
       dispatch,
       params.projectName,
       artifact.db_key,
-      artifact.tag,
-      artifact.tree,
+      artifact.uid,
       handleRefresh,
       filters,
       artifactType,
@@ -88,8 +87,7 @@ const DeleteArtifactPopUp = ({ artifact, artifactType, category, filters, handle
     })
   }, [
     artifact.db_key,
-    artifact.tag,
-    artifact.tree,
+    artifact.uid,
     artifactType,
     category,
     dispatch,

src/reducers/artifactsReducer.js

@@ -93,8 +93,8 @@ export const buildFunction = createAsyncThunk('buildFunction', ({ funcData }) =>
 })
 export const deleteArtifact = createAsyncThunk(
   'deleteArtifact',
-  ({ project, key, tag, tree, deletion_strategy, secrets }) => {
-    return artifactsApi.deleteArtifact(project, key, tag, tree, deletion_strategy, secrets)
+  ({ project, key, uid, deletion_strategy, secrets }) => {
+    return artifactsApi.deleteArtifact(project, key, uid, deletion_strategy, secrets)
   }
 )
 export const deleteArtifacts = createAsyncThunk(

src/utils/getUniqueIdentifier.js

@@ -34,9 +34,6 @@ export const getArtifactIdentifier = (artifact, unique) => {
   }
 
   if (unique) {
-    if (artifact?.tag) identifier += `.${artifact.tag}`
-    if (artifact?.tree) identifier += `.${artifact.tree}`
-    if (!isNaN(artifact?.iter)) identifier += `.${artifact.iter}`
     if (artifact?.uid) identifier += `.${artifact.uid}`
     if (artifact?.metadata?.uid) identifier += `.${artifact.metadata.uid}`
   }

src/utils/handleDeleteArtifact.js

@@ -27,8 +27,7 @@ export const handleDeleteArtifact = (
   dispatch,
   project,
   key,
-  tag,
-  tree,
+  uid,
   refreshArtifacts,
   filters,
   artifactType,
@@ -40,7 +39,7 @@ export const handleDeleteArtifact = (
   return dispatch(
     isDeleteAll
       ? deleteArtifacts({ project, name: key, category })
-      : deleteArtifact({ project, key, tag, tree, deletion_strategy, secrets })
+      : deleteArtifact({ project, key, uid, deletion_strategy, secrets })
   )
     .unwrap()
     .then(() => {
@@ -59,8 +58,7 @@ export const handleDeleteArtifact = (
           dispatch,
           project,
           key,
-          tag,
-          tree,
+          uid,
           refreshArtifacts,
           filters,
           artifactType,