@devtron-labs/devtron-fe-common-lib 0.0.10-beta-18 → 0.0.10-beta-19

package/dist/index.js

@@ -22788,7 +22788,7 @@ var ImageTagsContainer = function (_a) {
             lowercaseValue[0] == '.' ||
             lowercaseValue[0] == '-') {
             setTagErrorMessage('tag name cannot be empty or exceed 128 characters or cannot start with . or -');
-            return;
+            return false;
         }
         setTagErrorMessage('');
         var isTagExistsInExistingTags = existingTags.includes(lowercaseValue);
@@ -22799,24 +22799,26 @@ var ImageTagsContainer = function (_a) {
         }
         if (isTagExistsInExistingTags || isTagExistsInDisplayedTags || lowercaseValue === 'latest') {
             setTagErrorMessage('This tag is already being used in this application');
-            return;
+            return false;
         }
+        return true;
     };
     var handleTagCreate = function (newValue) {
         var lowercaseValue = newValue.toLowerCase().trim();
-        validateTag(lowercaseValue);
-        var newTag = {
-            id: 0,
-            tagName: lowercaseValue,
-            appId: 0,
-            deleted: false,
-            artifactId: 0,
-            duplicateTag: false,
-        };
-        setCreateTags(__spreadArray(__spreadArray([], __read(createTags), false), [newTag], false));
-        setDisplayedTags(__spreadArray(__spreadArray([], __read(displayedTags), false), [newTag], false));
-        setShowTagsWarning(true);
-        setTextInput('');
+        if (validateTag(lowercaseValue)) {
+            var newTag = {
+                id: 0,
+                tagName: lowercaseValue,
+                appId: 0,
+                deleted: false,
+                artifactId: 0,
+                duplicateTag: false,
+            };
+            setCreateTags(__spreadArray(__spreadArray([], __read(createTags), false), [newTag], false));
+            setDisplayedTags(__spreadArray(__spreadArray([], __read(displayedTags), false), [newTag], false));
+            setShowTagsWarning(true);
+            setTextInput('');
+        }
     };
     var handleTagSoftDelete = function (index) {
         var updatedTags = __spreadArray([], __read(displayedTags), false);
@@ -22916,13 +22918,10 @@ var ImageTagsContainer = function (_a) {
                     updateCurrentAppMaterial(artifactId, tags, (_j = res.result) === null || _j === void 0 ? void 0 : _j.imageComment);
             })
                 .catch(function (err) {
-                var _a, _b, _c, _d, _e, _f, _g;
+                var _a, _b, _c, _d, _e, _f;
                 // Fix toast message
                 if ((_d = (_c = (_b = (_a = err.errors) === null || _a === void 0 ? void 0 : _a[0]) === null || _b === void 0 ? void 0 : _b.userMessage) === null || _c === void 0 ? void 0 : _c.appReleaseTags) === null || _d === void 0 ? void 0 : _d.length) {
-                    var customError = __assign(__assign({}, err), { errors: [
-                            __assign(__assign({}, (_e = err.errors) === null || _e === void 0 ? void 0 : _e[0]), { userMessage: (_g = (_f = err.errors) === null || _f === void 0 ? void 0 : _f[0]) === null || _g === void 0 ? void 0 : _g.internalMessage }),
-                        ] });
-                    toast.error(customError);
+                    toast.error((_f = (_e = err.errors) === null || _e === void 0 ? void 0 : _e[0]) === null || _f === void 0 ? void 0 : _f.internalMessage);
                     errorStateHandling(err.errors);
                 }
                 else {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@devtron-labs/devtron-fe-common-lib",
-    "version": "0.0.10-beta-18",
+    "version": "0.0.10-beta-19",
     "description": "Supporting common component library",
     "main": "dist/index.js",
     "scripts": {

package/src/Common/ImageTags.tsx

@@ -115,7 +115,7 @@ export const ImageTagsContainer = ({
         setTagErrorMessage('')
     }
 
-    const validateTag = (lowercaseValue) => {
+    const validateTag = (lowercaseValue) : boolean => {
         if (
             lowercaseValue.length == 0 ||
             lowercaseValue.length >= 128 ||
@@ -123,7 +123,7 @@ export const ImageTagsContainer = ({
             lowercaseValue[0] == '-'
         ) {
             setTagErrorMessage('tag name cannot be empty or exceed 128 characters or cannot start with . or -')
-            return
+            return false
         }
         setTagErrorMessage('')
         const isTagExistsInExistingTags = existingTags.includes(lowercaseValue)
@@ -133,25 +133,27 @@ export const ImageTagsContainer = ({
         }
         if (isTagExistsInExistingTags || isTagExistsInDisplayedTags || lowercaseValue === 'latest') {
             setTagErrorMessage('This tag is already being used in this application')
-            return
+            return false
         }
+        return true
     }
 
     const handleTagCreate = (newValue) => {
         const lowercaseValue = newValue.toLowerCase().trim()
-        validateTag(lowercaseValue)
-        const newTag: ReleaseTag = {
-            id: 0,
-            tagName: lowercaseValue,
-            appId: 0,
-            deleted: false,
-            artifactId: 0,
-            duplicateTag: false,
+        if(validateTag(lowercaseValue)) {
+            const newTag: ReleaseTag = {
+                id: 0,
+                tagName: lowercaseValue,
+                appId: 0,
+                deleted: false,
+                artifactId: 0,
+                duplicateTag: false,
+            }
+            setCreateTags([...createTags, newTag])
+            setDisplayedTags([...displayedTags, newTag])
+            setShowTagsWarning(true)
+            setTextInput('')
         }
-        setCreateTags([...createTags, newTag])
-        setDisplayedTags([...displayedTags, newTag])
-        setShowTagsWarning(true)
-        setTextInput('')
     }
 
     const handleTagSoftDelete = (index) => {
@@ -259,16 +261,7 @@ export const ImageTagsContainer = ({
             .catch((err) => {
                 // Fix toast message
                 if (err.errors?.[0]?.userMessage?.appReleaseTags?.length) {
-                    const customError: ServerErrors = {
-                        ...err,
-                        errors: [
-                            {
-                                ...err.errors?.[0],
-                                userMessage: err.errors?.[0]?.internalMessage,
-                            },
-                        ],
-                    }
-                    toast.error(customError)
+                    toast.error(err.errors?.[0]?.internalMessage)
                     errorStateHandling(err.errors)
                 } else {
                     showError(err)